diff options
| author | Haru <haru@dotalux.com> | 2014-02-06 21:12:36 +0100 |
|---|---|---|
| committer | Haru <haru@dotalux.com> | 2014-03-05 22:35:03 +0100 |
| commit | a1918e7e9d1585587be270183866797eb1c682f5 (patch) | |
| tree | d07b30baaed8a40778c2510b5a112838eb1da7e5 /src/map/script.c | |
| parent | 4f3156b78f807553b11f37a7d0c898f6e358495b (diff) | |
| download | hercules-a1918e7e9d1585587be270183866797eb1c682f5.tar.gz hercules-a1918e7e9d1585587be270183866797eb1c682f5.tar.bz2 hercules-a1918e7e9d1585587be270183866797eb1c682f5.tar.xz hercules-a1918e7e9d1585587be270183866797eb1c682f5.zip | |
Corrected same-reference check to work on vars with the same name in different scopes
- Fixes bugreport:8008, special thanks to KeyWorld
http://hercules.ws/board/tracker/issue-8008-copyarray-fail-on-a-referenced-array-argument/
Signed-off-by: Haru <haru@dotalux.com>
Diffstat (limited to 'src/map/script.c')
| -rw-r--r-- | src/map/script.c | 25 |
1 files changed, 11 insertions, 14 deletions
diff --git a/src/map/script.c b/src/map/script.c index bf6dd448a..bef5f37e0 100644 --- a/src/map/script.c +++ b/src/map/script.c @@ -5770,30 +5770,27 @@ BUILDIN(copyarray) count = script_getnum(st, 4); if( count > SCRIPT_MAX_ARRAYSIZE - idx1 ) count = SCRIPT_MAX_ARRAYSIZE - idx1; - if( count <= 0 || (id1 == id2 && idx1 == idx2) ) + if( count <= 0 || (idx1 == idx2 && is_same_reference(data1, data2)) ) return true;// nothing to copy - if( id1 == id2 && idx1 > idx2 ) - {// destination might be overlapping the source - copy in reverse order - for( i = count - 1; i >= 0; --i ) - { + if( is_same_reference(data1, data2) && idx1 > idx2 ) { + // destination might be overlapping the source - copy in reverse order + for( i = count - 1; i >= 0; --i ) { v = script->get_val2(st, reference_uid(id2, idx2 + i), reference_getref(data2)); script->set_reg(st, sd, reference_uid(id1, idx1 + i), name1, v, reference_getref(data1)); script_removetop(st, -1, 0); } - } - else - {// normal copy - for( i = 0; i < count; ++i ) - { - if( idx2 + i < SCRIPT_MAX_ARRAYSIZE ) - { + } else { + // normal copy + for( i = 0; i < count; ++i ) { + if( idx2 + i < SCRIPT_MAX_ARRAYSIZE ) { v = script->get_val2(st, reference_uid(id2, idx2 + i), reference_getref(data2)); script->set_reg(st, sd, reference_uid(id1, idx1 + i), name1, v, reference_getref(data1)); script_removetop(st, -1, 0); - } - else// out of range - assume ""/0 + } else { + // out of range - assume ""/0 script->set_reg(st, sd, reference_uid(id1, idx1 + i), name1, (is_string_variable(name1)?(void*)"":(void*)0), reference_getref(data1)); + } } } return true; |