cuongdd2 / pw-tools

Automatically exported from code.google.com/p/pw-tools
0 stars 0 forks source link

sELedit uses hardcoded list indexes to perform several actions #2

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Reported by Necromaniak, Feb 2, 2011
I've never seen any content in lists 25, 51 and 52. So I would like to remove 
them by skipping 4 and 8 bytes in config. However this is currently impossible 
because the editor will breaks on loading because the dialogue list moved.

Additionally even if that worked, other features would break like the property 
replace.

Suggested fix: add ability to give lists an alias (by adding @<alias> after the 
name or any other mean), use this alias to perform list-specific actions.
For example:
> Dialogues
 @list58
> 0

> RAW

> byte:AUTO
Also, maybe fallback on numeric indexes if an alias wasn't found as it allows 
greater backward compatibility.

Comment 1 by Necromaniak, Feb 2, 2011
Just thought of another solution: config file could have a second part with all 
the info needed regarding list indexes and offsets. For example

Dialogue:58;
AiExport:38:0,2,64;

Original issue reported on code.google.com by max_schm...@gmx.net on 2 Sep 2012 at 9:51

GoogleCodeExporter commented 9 years ago
variable list index for list 58 in configuration files has been added

empty lists will not be removed since they are part of the structure

there is no need to implement the skill,property,... replace with a variable 
index since empty lists will not be dropped

Original comment by max_schm...@gmx.net on 2 Sep 2012 at 10:11

GoogleCodeExporter commented 9 years ago

Original comment by max_schm...@gmx.net on 2 Sep 2012 at 10:12