On Thu, 2010-06-24 at 11:32 +0200, Stephan Bergmann wrote: > On 06/24/10 11:17, Bartosz wrote: > > Maybe we should change it to (or remove this macro): > > typedef bool FASTBOOL; > > Yes, best would certainly be to remove the typedef and change > occurrences of FASTBOOL to plain bool (watching out for potential > misuses that tunnel values other than true/false through a variable of > type FASTBOOL).
This is about a sal_Bool rather than a bool, but I shall raise the question anyway. It just happens that I was running OO under gdb, and the following output had already caught my attention. Breakpoint 1, connectivity::OSkipDeletedSet::moveAbsolute (this=0xa85faa14, _nPos=1, _bRetrieveData=244 'รด') at /home/terry/OOo_hacking/DEV300_m83/connectivity/source/commontools/TSkipDeletedSet.cxx:170 170 sal_Bool OSkipDeletedSet::moveAbsolute(sal_Int32 _nPos,sal_Bool _bRetrieveData Is the funny value of _bRetrieveData sufficient grounds to create an issue? Cheers, Terry. --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@openoffice.org For additional commands, e-mail: dev-h...@openoffice.org