Difference between revisions of "Coding Conventions"

From ScummVM :: Wiki
Jump to navigation Jump to search
(Added an initial simple porting guide)
 
Line 17: Line 17:
* Furthermore, some games existed in multiple versions, e.g. one for Windows and one for MacOS. In that case, you may have to detect and distinguish these versions and employ differnt reading calls, to compensate for endian differences in the game data files.
* Furthermore, some games existed in multiple versions, e.g. one for Windows and one for MacOS. In that case, you may have to detect and distinguish these versions and employ differnt reading calls, to compensate for endian differences in the game data files.


== Struct packing ==
Do *not* assume that structs are stored in a certain way in memory -- the layout of a struct in memory can vary a lot between platforms, and on most modern systems it will almost never be "packed". If you absolutely must use packed structs, do not just use some #pragma or __attribute__ (as that is not portable either). Instead, do the following:
* Before the struct(s) you need to be packed, insert
  #include "common/pack-start.h" // START STRUCT PACKING
* After the struct(s) you need to be packed, insert
  #include "common/pack-end.h" // END STRUCT PACKING
* At the "end" of each struct you need to be packed, insert the following between the closing } and the ; after it:
  PACKED_STRUCT
You may want to look at some files which already do that, e.g. look at <code>engines/scumm/boxes.cpp</code>


== File access ==
== File access ==
TODO: Talk about file reading/writing, SaveFile vs. File, FSNodes vs. Paths, etc. -- maybe wait until the SoC, though? *g*
TODO: Talk about file reading/writing, SaveFile vs. File, FSNodes vs. Paths, etc. -- maybe wait until the SoC, though? *g*

Revision as of 18:04, 1 July 2007

Work in progress: This page is meant to list all sorts of typical portability issues that people working on ScummVM have to keep in mind. We should probably link on some nice well-established web pages for that on the net, and only focus on the most frequent or maybe unusual (ScummVM specific) issues.

Language features

ScummVM is written in a subset of C++. Due to limitations of the C++ run-times on various platforms, the following features can't be used:

  • C++ exceptions (throw/catch)
  • C++ RTTI (run-time type information, as in dynamic_cast<>)

Furthermore, the standard C++ library is a big no-no. Besides usually heavily relying on the above mentioned features, it also sucks up rather more resources than we'd like to, so we have our own replacements for various container classes etc.

We are reviewing these decisions from time to time, but so far, in our estimation the drawbacks of using any of these outweigh the hypothetical advantages.

Endianess issues

If you don't know what "Endianess issues" refers to, read up here: http://en.wikipedia.org/wiki/Endianess

ScummVM engine authors have to keep endianess issues in mind for two reasons:

  • ScummVM runs on both little endian (Windows, Intel Mac OS X, Intel Linux, ...) and big end hosts (PowerPC Mac OS X, ...). So when writing data (think savegames) to files and reading it back again, you need to compensate for this. This is easily done by using the READ_ and WRITE_ macros from common/endian.h (like READ_LE_UINT32 or WRITE_BE_UINT16.) resp. the corresponding Stream class methods (like readUint32LE or writeUint16BE)
  • Furthermore, some games existed in multiple versions, e.g. one for Windows and one for MacOS. In that case, you may have to detect and distinguish these versions and employ differnt reading calls, to compensate for endian differences in the game data files.

Struct packing

Do *not* assume that structs are stored in a certain way in memory -- the layout of a struct in memory can vary a lot between platforms, and on most modern systems it will almost never be "packed". If you absolutely must use packed structs, do not just use some #pragma or __attribute__ (as that is not portable either). Instead, do the following:

  • Before the struct(s) you need to be packed, insert
 #include "common/pack-start.h"	// START STRUCT PACKING
  • After the struct(s) you need to be packed, insert
 #include "common/pack-end.h"	// END STRUCT PACKING
  • At the "end" of each struct you need to be packed, insert the following between the closing } and the ; after it:
 PACKED_STRUCT

You may want to look at some files which already do that, e.g. look at engines/scumm/boxes.cpp

File access

TODO: Talk about file reading/writing, SaveFile vs. File, FSNodes vs. Paths, etc. -- maybe wait until the SoC, though? *g*