[#61424] [REJECT?] xmalloc/xfree: reduce atomic ops w/ thread-locals — Eric Wong <normalperson@...>

I'm unsure about this. I _hate_ the extra branches this adds;

13 messages 2014/03/12

[ruby-core:61333] [ruby-trunk - Bug #9582] [Rejected] CSV#headers not giving expected results.

From: jeg2@...
Date: 2014-03-06 16:26:50 UTC
List: ruby-core #61333
Issue #9582 has been updated by James Gray.

Status changed from Open to Rejected

Yeah, I also looked at making this change, but the behavior of returning just the first line is documented:

https://github.com/ruby/ruby/blob/trunk/lib/csv.rb#L672

I'm not sure if users of this library count on that or not.  This change would alter the behavior of tables, so at the very least I think it would need to happen on a major version release.

Tables are also editable, so deleting rows affects the results of what this method returns.

Anyway, if you need the non-first row headers, you can always do:

header_row = table.find { |row| !row.headers.empty? }
headers    = header_row && header_row.headers

----------------------------------------
Bug #9582: CSV#headers not giving expected results.
https://bugs.ruby-lang.org/issues/9582#change-45657

* Author: arup rakshit
* Status: Rejected
* Priority: Normal
* Assignee: 
* Category: 
* Target version: 
* ruby -v: -
* Backport: 
----------------------------------------
It seems something not working as expected in CSV. I tried to find the reason here , http://rxr.whitequark.org/mri/source/lib/csv.rb but I failed.

    require 'csv'
    
    content = <<_
    key,fr
    edit,テ嬰iter
    close,Fermer
    _
    
    File.write('test',content)
    
    file = CSV.read('test', :headers => true)
    file.headers # => ["key", "fr"]
    file.to_a # => [["key", "fr"], ["edit", "テ嬰iter"], ["close", "Fermer"]]
    
    content = <<_
    key,fr
    
    edit,テ嬰iter
    close,Fermer
    _
    
    File.write('test1',content)
    
    file = CSV.read('test1', :headers => true)
    file.headers # => []  # <~~~~~~~~~~~ Is this a bug ?
    file.to_a # => [[], [], ["edit", "テ嬰iter"], ["close", "Fermer"]]

---Files--------------------------------
9582.patch (870 Bytes)


-- 
http://bugs.ruby-lang.org/

In This Thread

Prev Next