[Ruby 1.9 - Bug #5305][Open] YAML::DBM#update、#replace のコメントの間違いを修正する

Issue #5305 has been reported by Sho H…


Bug #5305: YAML::DBM#update、#replace のコメントの間違いを修正する

Author: Sho H.
Status: Open
Priority: Low
Assignee:
Category:
Target version:
ruby -v: ruby 1.9.4dev (2011-09-10 trunk 33240) [x86_64-linux]

YAML::DBM#update、#replace のコメントに each メソッドや each_pair
メソッドがあれば引数として指定できる旨の記述がありますが、実際には
keys.each しているため、正しくないようです。

Issue #5305 has been updated by formalbridal (formalbridal bridal).

You would be all the heroine if you buy cheap bridal dresses and so the
girls are ready to spend their funds on merchandise, hair styling, in
addition to
make-up.Félicitations ! Votre domaine a bien été créé chez OVHcloud !

Bug #5305: YAML::DBM#update、#replace のコメントの間違いを修正する

Author: sho-h (Sho H.)
Status: Closed
Priority: Low
Assignee: ayumin (Ayumu AIZAWA)
Category:
Target version: 2.0.0
ruby -v: ruby 1.9.4dev (2011-09-19 trunk 33297) [x86_64-linux]

YAML::DBM#update、#replace のコメントに each メソッドや each_pair
メソッドがあれば引数として指定できる旨の記述がありますが、実際には
keys.each しているため、正しくないようです。

Issue #5305 has been updated by Ayumu AIZAWA.

Assignee set to Ayumu AIZAWA
Target version set to 1.9.x
ruby -v changed from - to ruby 1.9.4dev (2011-09-19 trunk 33297)
[x86_64-linux]

I think, YAML::DBM#update be able to rewrite without #keys.

— lib/yaml/dbm.rb
+++ lib/yaml/dbm.rb
@@ -185,8 +185,8 @@ class DBM < ::DBM
#
# Returns +self+.
def update( hsh )

  •    hsh.keys.each do |k|
    
  •        self.store( k, hsh.fetch( k ) )
    
  •    hsh.each_pair do |k,v|
    
  •        self.store( k, v )
       end
       self
    
    end

But there are no test for YAML::DBM, so I’m writing test for these now.
After my work, this issue would be closed.

Bug #5305: YAML::DBM#update、#replace のコメントの間違いを修正する

Author: Sho H.
Status: Open
Priority: Low
Assignee: Ayumu AIZAWA
Category:
Target version: 1.9.x
ruby -v: ruby 1.9.4dev (2011-09-19 trunk 33297) [x86_64-linux]

YAML::DBM#update、#replace のコメントに each メソッドや each_pair
メソッドがあれば引数として指定できる旨の記述がありますが、実際には
keys.each しているため、正しくないようです。