Project

General

Profile

Actions

Bug #17719

closed

Irregular evaluation order in hash literals

Added by nobu (Nobuyoshi Nakada) over 3 years ago. Updated about 3 years ago.

Status:
Closed
Assignee:
-
Target version:
-
[ruby-core:102831]

Description

@mame (Yusuke Endoh) pointed out an irregular evaluation order example.

$ ruby -e '{foo:p(1), bar:p(2), foo:p(3)}'
-e:1: warning: key :foo is duplicated and overwritten on line 1
1
3
2

It feels like a bug.
https://github.com/nobu/ruby/tree/duplicated-keys-order

Or, probably it would be better to turn into an error?

Updated by xtkoba (Tee KOBAYASHI) over 3 years ago

To me the evaluation order of 1->3->2 is a bit unexpected, but it might be OK unless specified otherwise.

IMO, duplicate keys in hash literals should be allowed in a scripting language like Ruby. For example, they are explicitly allowed in another scripting language named Python [1]. On the other hand, compiled languages should disallow them.

[1] https://mail.python.org/pipermail/python-ideas/2019-March/055726.html

Updated by nobu (Nobuyoshi Nakada) over 3 years ago

I think it can be fixed by removing the optimization in the parser and leaving it to the compiler.

Updated by jeremyevans0 (Jeremy Evans) about 3 years ago

nobu (Nobuyoshi Nakada) wrote in #note-2:

I think it can be fixed by removing the optimization in the parser and leaving it to the compiler.

I tried the approach of removing the optimization from the parser completely (https://github.com/jeremyevans/ruby/commit/bead10831e4f7ea6fc517fe66796f12579915fb3), but it doesn't handle duplicate keys in keyword arguments, resulting in incorrect unknown keyword errors: https://github.com/jeremyevans/ruby/runs/3900240357#step:15:168

I also tried rebasing your duplicated-keys-order branch against master, and that appears to fix the issue. I submitted a pull request for that: https://github.com/ruby/ruby/pull/4969

Actions #4

Updated by jeremyevans (Jeremy Evans) about 3 years ago

  • Status changed from Open to Closed

Applied in changeset git|fac2c0f73cafb5d65bfbba7aa8018fa427972d71.


Fix evaluation order of hash values for duplicate keys

Fixes [Bug #17719]

Co-authored-by: Nobuyoshi Nakada
Co-authored-by: Ivo Anjo

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0