Author: muggenhor
Date: Wed Feb 11 23:00:47 2009
New Revision: 6634

URL: http://svn.gna.org/viewcvs/warzone?rev=6634&view=rev
Log:
Major (but partly) cleanup of actionUpdateDroid:
 * Move variables into a more local scope
  - I don't think I screwed up start-at-block-declaration rules for MSVC, but 
given the size of these changes I can't be sure.
 * Remove a large amount of low level bit manipulation and replace it with 
arrays of boolean values instead
  - Yes bit shift and bitwise and where used; bitwise or was replaced with 
addition though
 * Don't use *WORD/*BYTE when we don't need the size guarantee (and the size 
guarantee is in fact an optimisation prone to introduce bugs)
 * "Concatenate" nested if statements into a single condition in a single 
if-statement
  - Less nested code to convey exactly the same meaning
 * Remove unused variable moveAction
  - It was only ever written to, but never read from

We don't need the size guarantee for WEAPON::nStat, so substitute "unsigned 
int" for "UDWORD"

Modified:
    trunk/src/action.c
    trunk/src/weapondef.h


_______________________________________________
Warzone-commits mailing list
[email protected]
https://mail.gna.org/listinfo/warzone-commits

Reply via email to