[sldev] BOOL patch, and test results.

Alan Grimes agrimes at speakeasy.net
Wed May 16 18:30:16 PDT 2007


>> Note 3: All instances of BOOL should be replaced with bool in all
>> functions, methods, and global data objects without hesitation or delay.

> Patch or GTFO. :)

Here it is.

While I have no intention of maintaining this patch separately from the
mainline, it does indeed work.

Changes:

- TRUE and FALSE have been removed and replaced with true and false as
per "The C++ Programming Language" by Bjarne Stroustrup

- Many usages of BOOL have been removed. (About half the work remains)

- Duplicate code has been removed, there were at least two instances of
trivial methods being implemented once for bool and again for BOOL.

- Declaration/implementation conflicts have been fixed. The compiler was
accepting conflicts between S32 declarations and BOOL implementations
because they were the same type. These have been fixed in this release.

- Linux build patches are included.

Testing has been carried out on the resulting binary. It performs BETTER
THAN ANY PREVIOUS ATTEMPT AT COMPILING THIS CODEBASE!! The only visual
glitch is that the "Vast Ocean" is not rendered, Otherwise this build is
very usable.

Due to the quantity of changes, I had to compress the file for
reasonable transmission on the ML. My appologies if this causes
difficulties.

-- 
Opera: Sing it loud! :o(  )>-<
-------------- next part --------------
A non-text attachment was scrubbed...
Name: boolpatch1.bz2
Type: application/octet-stream
Size: 751587 bytes
Desc: not available
Url : http://lists.secondlife.com/pipermail/sldev/attachments/20070516/5e6b5e84/boolpatch1-0001.obj


More information about the SLDev mailing list