ianpatt wrote:Sorry about this - I noticed he had been working on this for a while, and assumed it was happening with the knowledge and support of you guys, otherwise I would have spoken up on the list. I didn't look closely at the change enough.
No problem, I sort of knew that he was working on it, I'd seen one of his posts on the mw forums a few months back. I didn't think he'd ever submit it though, I thought he was just working on it for his own benefit to get better at C and add features he'd like for his own builds.
Some changes he has sent in are good, mainly converting function/variable names to more closely match the standards used in the rest of MAME. I question whether or not we need yet another new format, as I didn't see anything there that couldn't be fit in to the existing fields.
I admit I quite like the sound of some of the comment related changes, but most of the other features he's added I'm not sure of because I don't really understand what they do. Not be able to actually test the new features in u3 I suppose is blinkering my view that it's a steaming POS. The fact is I was more than happy with the cheat engine as it was, if the UI is as different as I think it is (I've deleted the u3 compile and recompiled it with the u2 cheat engine) then I'm really not very happy.
The crash when loading psx games is trivial to fix; if he had submitted the previous version of the file it wouldn't have happened. The old behavior of the cheat engine when running in to huge memory regions was to try the allocation, then skip the region if it failed. Now it asserts, killing MAME.
More importantly, the new cheat file loader doesn't even work for games with names shorter than 8 characters. That's a pretty big issue to overlook. There are several other issues I found as well (cpu_needs_swap always returns true after an extremely strange change, so writes of non-8-bit values are screwed up on some CPUs). All of these are relatively trivial things that should have come up in basic testing.
After looking at this more closely, I would have bounced the checkin, I just made a bad assumption that this was done in coordination with the people who actually find cheats and use the system heavily.
I'm not sure what will happen now, but I will discuss things on the mailing list.
Fingers crossed all the bugs will get fixed for u4, I don't know what the concensus of opinion is on the mailing list (not on it) but I imagine most devs couldn't give two hoots apart from the fact it maybe more compliant with the mame standards.