+1 for removing this ancient thing. Not for memory reasons (I doubt it would really make a difference), but for code cleanup reasons.
I don't know how risky this is when it comes to the binfilter module. If the old binary filters simply stream FASTBOOL variables, the document format would become incompatible. But IIRC we seldom used stream operators in our filters, but used SvStream::Read/Write<Type>(...), which would be fine then. So you should first check if this is the case... Malte. Stephan Bergmann wrote, On 06/24/10 11:32: > 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). > > -Stephan > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
