[#43120] [ruby-trunk - Bug #6124][Open] What is the purpose of "fake" gems in Ruby — Vit Ondruch <v.ondruch@...>

27 messages 2012/03/07

[#43142] Questions about thread performance (with benchmark included) — Rodrigo Rosenfeld Rosas <rr.rosas@...>

A while ago I've written an article entitled "How Nokogiri and JRuby

10 messages 2012/03/08

[#43148] [ruby-trunk - Feature #6126][Open] Introduce yes/no constants aliases for true/false — Egor Homakov <homakov@...>

16 messages 2012/03/09

[#43238] [ruby-trunk - Feature #6130][Open] inspect using to_s is pain — Thomas Sawyer <transfire@...>

21 messages 2012/03/11

[#43313] [ruby-trunk - Feature #6150][Open] add Enumerable#grep_v — Suraj Kurapati <sunaku@...>

17 messages 2012/03/15

[#43325] [ruby-trunk - Bug #6154][Open] Eliminate extending WaitReadable/Writable at runtime — Charles Nutter <headius@...>

25 messages 2012/03/16

[#43334] [ruby-trunk - Bug #6155][Open] Enumerable::Lazy#flat_map raises an exception when an element does not respond to #each — Dan Kubb <dan.kubb@...>

9 messages 2012/03/16

[#43370] [ruby-trunk - Feature #6166][Open] Enumerator::Lazy#pinch — Thomas Sawyer <transfire@...>

15 messages 2012/03/17

[#43373] [ruby-trunk - Bug #6168][Open] Segfault in OpenSSL bindings — Nguma Abojo <git.email.address@...>

14 messages 2012/03/17

[#43454] [ruby-trunk - Bug #6174][Open] Fix collision of ConditionVariable#wait timeout and #signal (+ other cosmetic changes) — "funny_falcon (Yura Sokolov)" <funny.falcon@...>

10 messages 2012/03/18

[#43497] [ruby-trunk - Bug #6179][Open] File::pos broken in Windows 1.9.3p125 — "jmthomas (Jason Thomas)" <jmthomas@...>

24 messages 2012/03/20

[#43502] [ruby-trunk - Feature #6180][Open] to_b for converting objects to a boolean value — "AaronLasseigne (Aaron Lasseigne)" <aaron.lasseigne@...>

17 messages 2012/03/20

[#43529] [ruby-trunk - Bug #6183][Open] Enumerator::Lazy performance issue — "gregolsen (Innokenty Mikhailov)" <anotheroneman@...>

36 messages 2012/03/21

[#43543] [ruby-trunk - Bug #6184][Open] [BUG] Segmentation fault ruby 1.9.3p165 (2012-03-18 revision 35078) [x86_64-darwin11.3.0] — "Gebor (Pierre-Henry Frohring)" <frohring.pierrehenry@...>

8 messages 2012/03/21

[#43672] [ruby-trunk - Feature #6201][Open] do_something then return :special_case (include "then" operator) — "rosenfeld (Rodrigo Rosenfeld Rosas)" <rr.rosas@...>

12 messages 2012/03/26

[#43678] [ruby-trunk - Bug #6203][Open] Array#values_at does not handle ranges with end index past the end of the array — "ferrous26 (Mark Rada)" <markrada26@...>

15 messages 2012/03/26

[#43794] [ruby-trunk - Feature #6216][Open] SystemStackError backtraces should not be reduced to one line — "postmodern (Hal Brodigan)" <postmodern.mod3@...>

15 messages 2012/03/28

[#43814] [ruby-trunk - Feature #6219][Open] Return value of Hash#store — "MartinBosslet (Martin Bosslet)" <Martin.Bosslet@...>

20 messages 2012/03/28

[#43858] [ruby-trunk - Feature #6222][Open] Use ++ to connect statements — "gcao (Guoliang Cao)" <gcao99@...>

12 messages 2012/03/29

[#43904] [ruby-trunk - Feature #6225][Open] Hash#+ — "trans (Thomas Sawyer)" <transfire@...>

36 messages 2012/03/29

[#43951] [ruby-trunk - Bug #6228][Open] [mingw] Errno::EBADF in ruby/test_io.rb on ruby_1_9_3 — "jonforums (Jon Forums)" <redmine@...>

28 messages 2012/03/30

[#43996] [ruby-trunk - Bug #6236][Open] WEBrick::HTTPServer swallows Exception — "regularfry (Alex Young)" <alex@...>

13 messages 2012/03/31

[ruby-core:43837] [ruby-trunk - Feature #6219] Return value of Hash#store

From: "shyouhei (Shyouhei Urabe)" <shyouhei@...>
Date: 2012-03-29 02:44:29 UTC
List: ruby-core #43837
Issue #6219 has been updated by shyouhei (Shyouhei Urabe).


=begin

Hmm, here you are a patch (not tested though).

 From c55a9c9fab30d51be77821bce36054fe365b49af Mon Sep 17 00:00:00 2001
 Message-Id: <c55a9c9fab30d51be77821bce36054fe365b49af.1332988729.git.shyouhei@ruby-lang.org>
 From: URABE, Shyouhei <[email protected]>
 Date: Thu, 29 Mar 2012 11:38:17 +0900
 Subject: [PATCH 1/1] Hash#store to return former content [feature #6219]
 
 Signed-off-by: URABE, Shyouhei <[email protected]>
 
 diff --git a/hash.c b/hash.c
 index e9937ff..0cd4e43 100644
 --- a/hash.c
 +++ b/hash.c
 @@ -1099,6 +1099,20 @@ copy_str_key(st_data_t str)
      return (st_data_t)rb_str_new4((VALUE)str);
  }
  
 +struct hash_aset_tuple {
 +    st_data_t old;
 +    st_data_t new;
 +};
 +
 +static int
 +hash_aset_i(st_data_t key, st_data_t *value, st_data_t arg)
 +{
 +    struct hash_aset_tuple *ptr = (void *)arg;
 +    ptr->old = *value;
 +    *value = ptr->new;
 +    return ST_CONTINUE;
 +}
 +
  /*
   *  call-seq:
   *     hsh[key] = value        -> value
 @@ -1120,15 +1134,19 @@ copy_str_key(st_data_t str)
  VALUE
  rb_hash_aset(VALUE hash, VALUE key, VALUE val)
  {
 +    struct hash_aset_tuple t;
      rb_hash_modify(hash);
      hash_update(hash, key);
 +    t.new = val;
 +    t.old = Qundef;
      if (RHASH(hash)->ntbl->type == &identhash || rb_obj_class(key) != rb_cString) {
 -	st_insert(RHASH(hash)->ntbl, key, val);
 +	st_update(RHASH(hash)->ntbl, key, hash_aset_i, (st_data_t)&t);
      }
      else {
 -	st_insert2(RHASH(hash)->ntbl, key, val, copy_str_key);
 +	st_data_t k = copy_str_key(key);
 +	st_update(RHASH(hash)->ntbl, k, hash_aset_i, (st_data_t)&t);
      }
 -    return val;
 +    return t.old;
  }
  
  static int
 -- 
 1.7.0.4

=end

----------------------------------------
Feature #6219: Return value of Hash#store
https://bugs.ruby-lang.org/issues/6219#change-25347

Author: MartinBosslet (Martin Bosslet)
Status: Open
Priority: Low
Assignee: 
Category: core
Target version: 


Hash#store returns the value that was just assigned, for example:

  h[:a] = b # => b

Does anyone rely on this behavior, are there cases when this becomes handy?

If however the return value is discarded most of the time, I was thinking it 
might be beneficial if we would return the previous value of a given key (nil if
none was assigned yet) instead. That way we could assign and check for a collision
in one pass, something that right now can only be done in two separate steps.




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

In This Thread