Csabo
Moderator
Posts: 541
Registered: 02-02 |
The quickfind box just locates the first matching lump and sets the focus to it. It definitely does not modify the WAD in any way. However, when a lump is clicked on (or selected through the quickfind), that's the first time XWE checks for the file modified date. So chances are, the file is already modified at that point. To better prove this (and to further help file corruption), maybe I should add a timer that would perform this check every one second or so in the background.
I tried to do all of the stuff you described but could not get the file modified warning to come up. I did find a few smaller bugs though that gave I/O Error 6. (That means invalid file handle btw, the most common cause is trying to work on a file that's not open.) For example, if you closed a WAD in XWE, you could still double-click the filter buttons, and it tried to import stuff. I fixed these and uploaded the latest beta.
For the last issue you described: in this case too, it really looks like the file was actually modified. XWE right now just gives you a warning and lets you continue, but maybe it shouldn't. You see, what happens is this: XWE reads the WAD file. The WAD file has a directory that specifies where the lumps are within the file. Let's say your DECORATE lump is at file position $1000. At some point XWE detects that the modified date of this WAD is newer than it should be, so it gives the warning. In the "new" file, DECORATE could be at $2000, or anywhere else. So when you clicked OK twice, and saw empty lumps, that's why it happened: XWE still looked for your lump at $1000, but the file was modified and now something else (or nothing) was at that position.
So thanks for your continued feedback, I really hope we can nail this. For now just grab the new one and see if you get it again.
For everyone else: could you guys please try to reproduce the file-gets-corrupted-on-quickfind-if-textures-lump-is-open issue? It doesn't happen to me, regardless of what I search for, or if I modify the textures lump or not.
|