On Wed, 2010-11-17 at 18:12 -0500, Kevin Hunter wrote:
> Hullo List,
>
> Is there a reason for lines that look like this?
>
> [...]
> /*N*/ String aGraphicId( rGraphicId );
> /*N*/ BfGraphicObject aGrfObject( ByteString( aGraphicId,
> RTL_TEXTENCODING_ASCII_US ) );
> /*N*/
> /*N*/ maTmp.EnableKillingFile();
> /*N*/
> /*N*/ if( aGrfObject.GetType() != GRAPHIC_NONE )
> /*N*/ {
> [...]
>
> The code is compiled, but what was the initial reason to have them in there?
To continue to be able to import/export the older native binary format,
which was closely connected to the old internal layout and structure of
the apps, but to allow decoupling of the apps from the older formats the
decision was made to create the binfilter as effectively copies of the
old sw/sd/starmath/sc apps, except stripped down to the bare minimum to
import and export the binary file formats and convert to/from the XML
file format.
So those /*N*/ comments mean something like "I think this is Necessary"
while there will be other places with e.g. /*?*/ meaning "Dunno, maybe
necessary"
C.
_______________________________________________
LibreOffice mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice
