[GSOC]Firebird Integration Weekly Update 03
nthiebaud at gmail.com
Sun Jun 30 22:17:41 PDT 2013
On Sun, Jun 30, 2013 at 2:05 PM, Andrzej J. R. Hunt <andrzej at ahunt.org> wrote:
> Finally I've had to do things like replace snprintf with _snprintf --
> currently decided using an ifdef -- this seems somewhat unclean but I'm
> not sure of any better way to do this.
Usually this is done in a header either by implementing a static
inline wrapper or sometime a simple #define does it.
#define snprintf _snprintf
The trick is to find the 'right' header to shove that in... or more
cleanly to have a separate 'compat' header and include it at the
judicious place, preferably an existing header that is already
included everywhere in the project...
The _last_ thing you want is to sprinkle
all over the code.
for one this is ugly, for second and more important, very fragile...
any new version of the underlying code taht introduce a new snprintf
would break and require a new patch
whereas with a header-based solution chance are good that it will
'just work' for future version.
More information about the LibreOffice