diff options
author | Jeremy Evans <code@jeremyevans.net> | 2021-06-08 10:19:08 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-06-08 10:19:08 -0700 |
commit | 117310bdc00236c0a7676616ce25b5106775dabc (patch) | |
tree | 3a9711e9446d22bca99c8a56e8ceff723f0c8f04 /hash.c | |
parent | 8c87efaa8a45166ed977294330c32a4b186b8e7b (diff) |
Make ENV.clone warn and ENV.dup raise
ENV.dup returned a plain Object, since all of ENV's behavior is
defined in ENV's singleton class. So using dup makes no sense.
ENV.clone works and is used in some gems, but it doesn't do what
the user expects, since modifying ENV.clone also modifies ENV.
Add a deprecation warning pointing the user to use ENV.to_h
instead.
This also undefines some private initialize* methods in ENV,
since they are not needed.
Fixes [Bug #17767]
Notes
Notes:
Merged: https://github.com/ruby/ruby/pull/4557
Merged-By: jeremyevans <code@jeremyevans.net>
Diffstat (limited to 'hash.c')
-rw-r--r-- | hash.c | 63 |
1 files changed, 63 insertions, 0 deletions
@@ -6488,6 +6488,61 @@ env_update(VALUE env, VALUE hash) } /* + * call-seq: + * ENV.clone(freeze: nil) -> copy of ENV + * + * Returns a clone of ENV, but warns because the ENV data is shared with the + * clone. + * If +freeze+ keyword is given and not +nil+ or +false+, raises ArgumentError. + * If +freeze+ keyword is given and +true+, raises TypeError, as ENV storage + * cannot be frozen. + */ +static VALUE +env_clone(int argc, VALUE *argv, VALUE obj) +{ + if (argc) { + static ID keyword_ids[1]; + VALUE opt, kwfreeze; + + if (!keyword_ids[0]) { + CONST_ID(keyword_ids[0], "freeze"); + } + rb_scan_args(argc, argv, "0:", &opt); + if (!NIL_P(opt)) { + rb_get_kwargs(opt, keyword_ids, 0, 1, &kwfreeze); + switch(kwfreeze) { + case Qtrue: + rb_raise(rb_eTypeError, "cannot freeze ENV"); + break; + default: + rb_raise(rb_eArgError, "invalid value for freeze keyword"); + break; + case Qnil: + case Qfalse: + break; + } + } + } + + rb_warn_deprecated("ENV.clone", "ENV.to_h"); + return envtbl; +} + +NORETURN(static VALUE env_dup(VALUE)); +/* + * call-seq: + * ENV.dup # raises TypeError + * + * Raises TypeError, because ENV is a singleton object. + * Use #to_h to get a copy of ENV data as a hash. + */ +static VALUE +env_dup(VALUE obj) +{ + rb_raise(rb_eTypeError, "Cannot dup ENV, use ENV.to_h to get a copy of ENV as a hash"); +} + +/* * A \Hash maps each of its unique keys to a specific value. * * A \Hash has certain similarities to an \Array, but: @@ -7193,6 +7248,14 @@ Init_Hash(void) rb_define_singleton_method(envtbl, "to_h", env_to_h, 0); rb_define_singleton_method(envtbl, "assoc", env_assoc, 1); rb_define_singleton_method(envtbl, "rassoc", env_rassoc, 1); + rb_define_singleton_method(envtbl, "clone", env_clone, -1); + rb_define_singleton_method(envtbl, "dup", env_dup, 0); + + VALUE envtbl_class = rb_singleton_class(envtbl); + rb_undef_method(envtbl_class, "initialize"); + rb_undef_method(envtbl_class, "initialize_clone"); + rb_undef_method(envtbl_class, "initialize_copy"); + rb_undef_method(envtbl_class, "initialize_dup"); /* * ENV is a Hash-like accessor for environment variables. |