aboutsummaryrefslogtreecommitdiff
path: root/engines/neverhood/modules/module2700.cpp
AgeCommit message (Collapse)Author
2015-10-18NEVERHOOD: Pass NPoint as position instead of X in two more casesTorbjörn Andersson
According to johndoe it's correct to pass the entire coordinate, rather than just X, to the car in these cases as well. I can't tell any difference at all in the behavior, but I guess it doesn't hurt either. After all, we can easily zero the Y coordinate, if we ever want to.
2015-10-11NEVERHOOD: Possible fix for bad car behaviourTorbjörn Andersson
This is something I found when trying the savegame from bug #6932, but I still don't know if it actually is that bug. From what I understand, there are two different cases in the moveCarToPoint() method: One where you click on a different section on a track than you're on, and one where you click on the same section on the track that you're on. In the latter case, it sends message 0x2004 to the car, which is then handled by AsCommonCar::handleMessage(). That one will assume that the parameter is a point, but this can also be encoded as an integer with 16 bits for the X coordinate and 16 bits for the Y coordinate. See MessageParam::asPoint(). If we only pass an X coordinate to the message, the Y coordinate is assumed to be 0, and we do this in a couple of places. I do not know the exact implications of that, but in the two cases I've changed here, it meant that clicking on the track below the car would still make it go up, because it thought you were travelling towards the top of the screen. So I think this is the appropriate fix, but even if it is, I do not know if it's enough or if it should be changed in other places as well.
2014-06-05NEVERHOOD: Fix radio part of bug #6513 ("Some nasty bugs")Torbjörn Andersson
This clears the radio music variable when leaving the room. I've discussed with johndoe123 and he says this is what the original does. (You could also get the same effect by clearing it when entering the room.)
2014-03-19NEVERHOOD: Fix some uninitialized variablesStrangerke
2014-02-18NEVERHOOD: Make GPL headers consisten in themselves.Johannes Schickel
2013-12-25NEVERHOOD: Move car to the correct position when entering a new sceneFilippos Karapetis
2013-12-25NEVERHOOD: Change more message values to enumsFilippos Karapetis
2013-12-25NEVERHOOD: Change more message values to enumsFilippos Karapetis
2013-10-05NEVERHOOD: Clean up some includesFilippos Karapetis
2013-10-05NEVERHOOD: Split sprites from their scenes in module 1600Filippos Karapetis
2013-10-05NEVERHOOD: Split sprites from their scenes in modules 2100 and 2200Filippos Karapetis
This also reduces inter-header dependencies
2013-10-01NEVERHOOD: Split some more sprites from their respective scenesFilippos Karapetis
This splits modules 2700, 2900 and 3000
2013-08-05NEVERHOOD: Remove NRect constructorMax Horn
Otherwise, every global variable of type NRect requires a constructor to be run, which can cause portability issue.
2013-07-14JANITORIAL: Remove trailing whitespaceSven Hesse
2013-07-03NEVERHOOD: Fix typo in variable nameFilippos Karapetis
2013-06-10NEVERHOOD: Change the debuglevel of the scene changing debug outputFilippos Karapetis
These are shown on each scene change, but should no longer be needed to be enabled by default, since we can now use the debugger to check and change the current scene
2013-05-08NEVERHOOD: Move module files to own subdirectoryjohndoe123