[#41916] Proposal: Bitmap Marking GC — Narihiro Nakamura <authornari@...>

Hi.

18 messages 2012/01/05

[#41941] [ruby-trunk - Bug #5851][Open] make check fails when compiling with GCC 4.7 - *** longjmp causes uninitialized stack frame *** — Vit Ondruch <v.ondruch@...>

12 messages 2012/01/06

[#41979] [ruby-trunk - Bug #5865][Open] Exception#== should return false if the classes differ — Hiro Asari <asari.ruby@...>

10 messages 2012/01/08

[#42003] [ruby-trunk - Bug #5871][Open] regexp \W matches some word characters when inside a case-insensitive character class — Gareth Adams <gareth@...>

14 messages 2012/01/09

[#42016] [ruby-trunk - Feature #5873][Open] Adopt FFI over DL — Heesob Park <phasis@...>

15 messages 2012/01/10

[#42149] [ruby-trunk - Feature #5899][Open] chaining comparsions. — Ondrej Bilka <neleai@...>

12 messages 2012/01/16

[#42164] [ruby-trunk - Feature #5903][Open] Optimize st_table (take 2) — Yura Sokolov <funny.falcon@...>

18 messages 2012/01/17

[ruby-core:41867] [Feature patch] Set#to_ary to play well with Arrays

From: Colin Scott <cs@...>
Date: 2012-01-02 01:21:46 UTC
List: ruby-core #41867
2012-12-01 Colin Scott <cs@cs.berkeley.edu>

* lib/set.rb (to_a): make to_ary an alias

Sets don't play very well with arrays. For example:

irb(main):001:0> require 'set'
=> true
irb(main):002:0> s = Set.new
=> #<Set: {}>
irb(main):003:0> a = []
=> []
irb(main):004:0> s | a
=> #<Set: {}>
irb(main):005:0> a | s
*TypeError: can't convert Set into Array*

Set#to_a is already defined. By aliasing Set#to_ary to Set#to_a, the type
conversion works. This should not change the behavior of Sets otherwise.

Attachments (1)

ruby-changes.patch (332 Bytes, text/x-diff)
Index: lib/set.rb
===================================================================
--- lib/set.rb	(revision 34167)
+++ lib/set.rb	(working copy)
@@ -145,6 +145,9 @@
     @hash.keys
   end
 
+  # Alias of to_a
+  alias :to_ary :to_a
+
   def flatten_merge(set, seen = Set.new) # :nodoc:
     set.each { |e|
       if e.is_a?(Set)

In This Thread

Prev Next