Age | Commit message (Collapse) | Author | |
---|---|---|---|
2014-03-10 | TONY: Fix clang warnings - Courtesy of LordHoto | Strangerke | |
2014-03-09 | TONY: Add some missing virtual keywords | Strangerke | |
2014-03-01 | TONY: Remove unused error defines | Strangerke | |
2014-03-01 | TONY: Remove dummy defines | Strangerke | |
2014-03-01 | TONY: Remove unused function declaration in lzo | Strangerke | |
2014-03-01 | TONY: Indent code properly in lzo | Strangerke | |
2014-03-01 | TONY: Remove dead code in lzo | Strangerke | |
2014-02-28 | TONY: Reduce the scope of more variables | Strangerke | |
2014-02-28 | TONY: Fix some British comments | Strangerke | |
2014-02-28 | TONY: Reduce the scope of some more variables | Strangerke | |
2014-02-28 | TONY: Reduce the scope of some variables, change the return type of findPath() | Strangerke | |
2014-02-18 | TONY: Make GPL headers consistent in themselves. | Johannes Schickel | |
2014-02-17 | TONY: Indent REGISTER_PLUGIN_* for consistency. | Johannes Schickel | |
2014-01-17 | Merge pull request #417 from digitall/STACK_fixes | Johannes Schickel | |
ALL: Fix optimization unstable code on checking for null after new. | |||
2014-01-16 | TONY: Cleanup (don't compare pointer with false). | Johannes Schickel | |
2014-01-15 | ALL: Remove optimization unstable code on checking for null after new. | D G Turner | |
These issues were identified by the STACK tool. By default, the C++ new operator will throw an exception on allocation failure, rather than returning a null pointer. The result is that testing the returned pointer for null is redundant and _may_ be removed by the compiler. This is thus optimization unstable and may result in incorrect behaviour at runtime. However, we do not use exceptions as they are not supported by all compilers and may be disabled. To make this stable without removing the null check, you could qualify the new operator call with std::nothrow to indicate that this should return a null, rather than throwing an exception. However, using (std::nothrow) was not desirable due to the Symbian toolchain lacking a <new> header. A global solution to this was also not easy by redefining "new" as "new (std::nothrow)" due to custom constructors in NDS toolchain and various common classes. Also, this would then need explicit checks for OOM adding to all new usages as per C malloc which is untidy. For now to remove this optimisation unstable code is best as it is likely to not be present anyway, and OOM will cause a system library exception instead, even without exceptions enabled in the application code. | |||
2013-12-17 | TONY: Add a reset function in RMWindow in order to group variable initialization | Strangerke | |
2013-12-07 | TONY: Add missing getDebugger() for engine API. | D G Turner | |
2013-11-24 | BUILD: Remove need for engine.mk in each engine directory. | D G Turner | |
Each engine now only has to provide a single configure.engine file adding the engine into the configure script, which then produces the required other files automatically. | |||
2013-11-24 | BUILD: Remove need for engine-plugin.h in engines. | D G Turner | |
This is now generated automatically by the configure script from the engine directory names. | |||
2013-11-24 | BUILD: Split engines/plugins_table header down to a file per engine. | D G Turner | |
This is the third and final commit enabling fully pluggable engines. Now providing an engine folder contains a configure.engine, engine.mk and engine-plugin.h file, it will be picked up automatically by the configure script. | |||
2013-11-24 | BUILD: Split engines.mk down to a single file per engine. | D G Turner | |
This is the second part of allowing engines to be added dynamically. Each folder in engines/ which must contain a file named "engine.mk" containing the make definitions for that engine. | |||
2013-11-24 | BUILD: Split configure.engines down to a single file per engine. | D G Turner | |
This is the first part of allowing engines to be added dynamically. They are placed into a folder in engines/ which must contain a file named "configure.engine" to add the engine, which is pulled into the top level configure script automatically. | |||
2013-11-23 | TONY: Janitorial - Fix spacing errors | Strangerke | |
2013-10-13 | TONY: Initialise _buf and _origBuf in second RMGfxBuffer constructor | Torbjörn Andersson | |
I think this fixes CID 1086888, but I haven't been able to find where in the game this constructor is used. | |||
2013-09-28 | TONY: Fix compilation on FreeBSD with C++-11 enabled. | D G Turner | |
This is part of bug #3615056 and is due to NULL being defined as nullptr when C++-11 is enabled, which is not valid being assigned to a integer value. | |||
2013-08-03 | TONY: Take advantage of Surface::getPixels. | Johannes Schickel | |
2013-08-03 | TONY: Prefer getBasePtr over direct Surface::pixels access. | Johannes Schickel | |
2013-08-02 | TONY: Add a hack to work around amigaos4 issues. | Alyssa Milburn | |
2013-08-02 | TONY: Fix thumbnails on BE. | Alyssa Milburn | |
2013-08-02 | TONY: Fix savegames on BE (this breaks compatibility). | Alyssa Milburn | |
2013-08-02 | TONY: Simplify thumbnail loading code. | Alyssa Milburn | |
2013-07-19 | TONY: Remove debug message from last commit. Oops. | Torbjörn Andersson | |
2013-07-19 | TONY: Fix parameter to RMGfxSourceBuffer8RLEWordAA::init() | Torbjörn Andersson | |
The method it is meant to override uses "const byte *buf" so this one should too. I don't know if/when it's called, but fuzzie tells me it should be a harmless change. Hopefully fixes CID 1003892. | |||
2013-06-11 | TONY: Add detection for German "Shoe Box", bug #3582420 | Strangerke | |
2013-06-11 | TONY: Add detection for English version unpacked | Strangerke | |
Thanks Tomaso for reporting the problem | |||
2013-06-10 | TONY: Remove varargs from mpalQueryCORO(). CID 1002105 | Torbjörn Andersson | |
It didn't always reach va_end(), but rather than adding a call it makes more sense to just drop the varargs from it, since it didn't use it anyway. | |||
2013-06-02 | TONY: Fix memory leak on save game failure (CID 1003579) | Torbjörn Andersson | |
Though shouldn't it also alert the user to the failure? | |||
2013-06-02 | TONY: Fix potential memory leak | Torbjörn Andersson | |
Unfortunately, I'm not sure where and when this function is called. But this should be correct. CID 1003575 | |||
2012-11-20 | TONY: Fix compiler warnings about possible "used before set" variables. | D G Turner | |
Looking at this code and the code in clip2D, this shouldn't happen, but this will prevent any future issues. | |||
2012-11-14 | TONY: Fixed scripted exit to use engine method, rather than error(). | D G Turner | |
This fixes bug #3586252 ("TONY: Game errors out when exiting via the menu"). | |||
2012-11-13 | TONY: Fix image loading on BE systems. | D G Turner | |
Thanks to wjp and fuzzie for working out the cause. | |||
2012-11-12 | TONY: Fix resource loading on BE systems | Willem Jan Palenstijn | |
Thanks to digitall for debugging and testing. | |||
2012-11-12 | TONY: Fix compiler warnings in Input class. | D G Turner | |
2012-11-12 | TONY: Cleanup whitespace formatting. | D G Turner | |
This is mainly removal of extraneous blank lines, reformatting indents to tab characters and other minor non-functional changes to improve compliance with Code Formatting Conventions. | |||
2012-11-11 | TONY: Replace Input class internal buffers with Common standard. | D G Turner | |
The mouse position is now represented internally by Common::Point and the keyDown buffer is an array of Common::Keycode values for depressed keys. | |||
2012-11-11 | TONY: Remove further unused code from Input class. | D G Turner | |
2012-11-11 | TONY: Remove unused functions from Input class. | D G Turner | |
2012-11-02 | TONY: Fix endianness issue when saving/loading inventory | Willem Jan Palenstijn | |
2012-11-01 | Merge pull request #290 from rasky/fix_sfx_volumes | Paul Gilbert | |
TONY: fix volumes of sound effects. |