Skip to content

rb_ary_dup_of_p - stricter change for array's copy #216

Closed
wants to merge 1 commit into from

2 participants

@funny-falcon
  • array.c (rb_ary_dup_of_p): new function
    stricter check for equality of previously dup-ed array,
    because rb_ary_shared_with_p doesn't handle case when #shift or #pop
    were applied to original array

  • include/ruby/intern.h (rb_ary_dup_of_p): declare

  • load.c (rb_get_expanded_load_path, get_loaded_features_index):
    use rb_ary_dup_of_p instead of rb_ary_shared_with_p

@funny-falcon funny-falcon rb_ary_dup_of_p - stricter change for array's copy
* array.c (rb_ary_dup_of_p): new function
  stricter check for equality of previously dup-ed array,
  because rb_ary_shared_with_p doesn't handle case when #shift or #pop
  were applied to original array

* include/ruby/intern.h (rb_ary_dup_of_p): declare

* load.c (rb_get_expanded_load_path, get_loaded_features_index):
  use rb_ary_dup_of_p instead of rb_ary_shared_with_p
07006ce
@zzak
Ruby Programming Language member
zzak commented Nov 19, 2012

Closing, please refer to Bug #7383 in redmine

@zzak zzak closed this Nov 19, 2012
@funny-falcon

@zzak what did you mean? I placed a link to that issue (which I created for referring this pull request)

@zzak
Ruby Programming Language member
zzak commented Nov 19, 2012

@funny-falcon Yes, the ticket on redmine is preferred since many committers don't use github.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.