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: [email protected]
For additional commands, e-mail: [email protected]
