raziel23x / skyrim-plugin-decoding-project

Automatically exported from code.google.com/p/skyrim-plugin-decoding-project
1 stars 3 forks source link

Voice Type EditorID disappears. #100

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

Seems to have no repeatable, consistent pattern.

What version of the product are you using? On what operating system?
3.0.26, FNVEdit, Windows 7

Please provide any additional information below.

When working on Voice Types, oft times the program no longer reads the EditorID 
subrecord, leaving that field blank, but in the record header it clearly shows 
the EditorID and FormID normally as if it had one. Worse still, this bug will 
be saved into the file when it is saved. This hasn't happened with any other 
record and appears to be limited - currently - to Voice Types. It's not that 
critical of a bug, since I can copy the EditorID from the record header by 
double clicking it and coping the ID from the info window, but it is extremely 
odd to have it happen. If I find the steps to reproduce I will redo this bug 
report.

Original issue reported on code.google.com by harkness...@yahoo.com on 15 Jan 2013 at 11:07

GoogleCodeExporter commented 9 years ago
We had that at skyrim too, there is a legacy bug that doesn't read the EDID 
subrecord if
1) record itself has only 2 subrecords(?) including EDID, not sure
2) they are both marked as required
3) the second subrecord is absent

This can happen only to very simple records, like Voice Type in your case. 
There is a workaround to that, unmark them as required which I'll do for 
FNVEdit. Or Hugues will fix it if he wants to.

Original comment by zila...@gmail.com on 16 Jan 2013 at 6:48

GoogleCodeExporter commented 9 years ago
I'll try to look at that. But anyway, if you have vanilla records without the 
second record it should not be required no?

Original comment by HuguesLe...@gmail.com on 16 Jan 2013 at 8:53

GoogleCodeExporter commented 9 years ago
Yes it (second subrecord) was marked as required by Elminster, but in reality 
it isn't because there are vanilla records without them. Just an error in 
record definition, however it should not skip EDID in that case, it is clearly 
a bug.

Original comment by zila...@gmail.com on 16 Jan 2013 at 11:36

GoogleCodeExporter commented 9 years ago
EditorIDs are required for every record except ACHR, ACRE, REFR, NAVM, and the 
NAVI. The second subrecrod in Voice Types are simply flags and aren't required 
for it to work properly.

Original comment by harkness...@yahoo.com on 16 Jan 2013 at 5:20

GoogleCodeExporter commented 9 years ago
Tried debugging that, but I have a hard time setting a breakpoint at the proper 
time.

Original comment by HuguesLe...@gmail.com on 20 Jan 2013 at 4:33

GoogleCodeExporter commented 9 years ago
Looks to have been solved in 3.0.29. Most likely by properly detecting the end 
of assigned memory.

Original comment by HuguesLe...@gmail.com on 24 Feb 2013 at 10:55

GoogleCodeExporter commented 9 years ago

Original comment by zila...@gmail.com on 19 Mar 2013 at 8:00