Thread (19 messages) flat view 19 messages, 3 authors, 2016-06-15

Re: [PATCH/RFC v7 1/2] Add infrastructure for translating Git with gettext

From: Jonathan Nieder <hidden>
Date: 2016-06-15 22:48:55

Ævar Arnfjörð Bjarmason wrote:
On Sat, Jun 5, 2010 at 03:01, Jonathan Nieder [off-list ref] wrote:
quoted
 #ifdef NO_GETTEXT
 static inline void git_setup_gettext(void) {}
 #endif

and

 ifndef NO_GETTEXT
 LIB_OBJS += gettext.o
 endif
Sure, but that would be putting code in a header file, which is
usually taboo. It looks like there's some prior art on that though.
Like strbuf.h.

I don't care either way, what do you think?
This is what ‘inline’ is for.  I think using it for stubs like this
is perfectly acceptable and improves readability.

Jonathan
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help