Details

    • Number of attachments :
      0

      Description

      CRuby devs also want to dump $, but it uses $, now.

      Ref. http://bugs.ruby-lang.org/issues/5915

      % ruby187 -e '$,="_"; p [1,2].join(nil)'
      "1_2"
      % ruby -e '$,="_"; p [1,2].join(nil)' 
      "1_2"
      % jruby -e '$,="_"; p [1,2].join(nil)'  
      "12"
      % rbx -e '$,="_"; p [1,2].join(nil)'  
      "1_2"
      % mruby -e '$,="_"; p [1,2].join(nil)' 
      "12"
      

        Activity

        Hide
        Hiroshi Nakamura added a comment -

        Update description for formatting.

        Show
        Hiroshi Nakamura added a comment - Update description for formatting.
        Hide
        Charles Oliver Nutter added a comment -

        It looks like the resolution was that the current behavior (in MRI) is correct, and the bug was a documentation issue. So I will fix to do what MRI does.

        Show
        Charles Oliver Nutter added a comment - It looks like the resolution was that the current behavior (in MRI) is correct, and the bug was a documentation issue. So I will fix to do what MRI does.
        Hide
        Charles Oliver Nutter added a comment -
        commit 39cbc8292ed7eca771fb51e04a34fdd166e71e72
        Author: Charles Oliver Nutter <headius@headius.com>
        Date:   Tue Jul 24 14:03:50 2012 -0500
        
            Fix JRUBY-6776
            
            Array#join(nil) should use $, as a separator for compatibility
        
        :100644 100644 ca24a0f... 485a2be... M	src/org/jruby/RubyArray.java
        
        Show
        Charles Oliver Nutter added a comment - commit 39cbc8292ed7eca771fb51e04a34fdd166e71e72 Author: Charles Oliver Nutter <headius@headius.com> Date: Tue Jul 24 14:03:50 2012 -0500 Fix JRUBY-6776 Array#join(nil) should use $, as a separator for compatibility :100644 100644 ca24a0f... 485a2be... M src/org/jruby/RubyArray.java

          People

          • Assignee:
            Charles Oliver Nutter
            Reporter:
            Hiroshi Nakamura
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: