Ticket #1160 (closed bug: fixed)

Opened 2 years ago

Last modified 23 months ago

properly handle different line endings

Reported by: lanz Owned by: lanz
Milestone: 2.6 Component: addon/bioinformatics
Severity: minor Keywords:
Cc: lanz Blocking:
Blocked By:

Description

Someone should grep orange and especially bioinformatics for .split("\n") and replace with .splitlines()

Change History

comment:1 Changed 2 years ago by lanz

  • Owner changed from lanz to marko
  • Status changed from new to assigned
  • Cc lanz added

Marko, I am reassigning this to you just to make sure you see this and will not use that split again :) (if it wasn't you I apologize)

But you don't have to do it, I can fix it next time I will need a break.

comment:2 Changed 2 years ago by lanz

  • Owner changed from marko to lanz
  • Status changed from assigned to accepted
  • Component changed from library to bioinformatics add-on

I did a find and replace on orange in [ac200cc97b30].

Bioinformatics has more of these...

comment:3 Changed 23 months ago by Lan Zagar <lan.zagar@…>

  • Status changed from accepted to closed
  • Resolution set to fixed

In [2e2c671e430092810a4744cb0cc94bee5507a4c3/orange-bioinformatics]:

Replaced split(\n) with splitlines (closes #1160).

Note: See TracTickets for help on using tickets.