[Libreoffice] Need a little help with the easyhack "Strip include guards in idl files"

Nuno J. Silva nunojsilva at ist.utl.pt
Tue Mar 29 03:47:59 PDT 2011


Nuno J. Silva <nunojsilva at ist.utl.pt> writes:
> Julien Nabet <serval2412 at yahoo.fr> writes:
>
>> Then I took a look at the directory "ure/offapi". There are about 3700
>> files and the strip-guards script lets a lot (more than 3000!) of
>> "#ifndef" with "#include".( I don't know why, a pb of Unix/Dos
>> end-of-line perhaps ?)
>> This time it's too much to do it manually.
>> Some idea to improve the strip-guards script ?
>
> I looked at your commit with the manually fixed guards[1] and at least
> some of these are explainable. Maybe that's why there are so many guards
> left in offapi:
>
> [1] http://cgit.freedesktop.org/libreoffice/ure/commit/?id=cab76775a72e2e9e6cd242bbf87dd23475999164
[..]
> - Some guards are written with lower case, while they're defined in
>   upper case in the included file, like
>  
>    #ifndef __com_sun_star_bridge_XBridge_idl__
>    #include <com/sun/star/bridge/XBridge.idl>
>
>   while the file does
>
>    #define _COM_SUN_STAR_BRIDGE_XBRIDGE_IDL_ 

... but of course here the issue is not only upper case, but also
missing underscores.

-- 
Nuno J. Silva (aka njsg)
gopher://sdf-eu.org/1/users/njsg


More information about the LibreOffice mailing list