You may be familiar with variadic functions in C, these are basically functions that allow a variable number of parameters, they are normally written like this:
void my_print_func(const char *format, ...)
{
va_list args;
va_start(args, format);
vprintf(format, args);
va_end(args);
}
Obviously you can do more with them and they are very useful, but if you want to have one function calling another things can get complicated. This post explores the problem and a couple of ways of solving it.
The core problem is that a va_list
can only be used once, if you try to do the following you are in for a bad time:
void my_print_func(const char *format, ...)
{
va_list args;
va_start(args, format);
vprintf(format, args);
va_end(args);
}
void my_print_func2(bool should_print, const char *format, ...)
{
if (!should_print)
{
return;
}
va_list args;
va_start(args, format);
my_print_func(format, args);
va_end(args);
}
This will likely compile, but your text will get quite scrambled along the way when it is executed. So, you have two options, the first is to pass the va_list
to the parent function like this:
void my_print_func(const char *format, va_list args)
{
vprintf(format, args);
}
void my_print_func2(bool should_print, const char *format, ...)
{
if (!should_print)
{
return;
}
va_list args;
va_start(args, format);
my_print_func(format, args);
va_end(args);
}
This is fine if you own my_print_func()
and the call chain is only going to work like this. But if, like me, you are in a scenario where my_print_func()
and my_print_func2()
could be called, what can you do? Well, this is where macros come in.
Basically we define my_print_func2()
as a macro which calls my_print_func()
, the macro can have a small amount of logic if needed. Note that you need to be compiling with at least C99 standard for this to work:
#define my_print_func2(should_print, format, ...)\
do {\
if (!should_print)\
{\
return;\
}\
else\
{\
my_print_func(format, __VA_ARGS__);\
}\
} while (0)
void my_print_func(const char *format, ...)
{
va_list args;
va_start(args, format);
vprintf(format, args); // Or any function that takes a va_list parameter
va_end(args);
}
Since the macro essentially fills in code at compile time to where it is used it is essentially just one call to my_print_func()
, but it acts like a separate call to the reader.
I recently acquired an Amiga 1200 motherboard in a spares/repairs condition for about £100 recently.…
Whilst I do like working with STM32 development boards, some basic information I need can…
When I last left this blog series, the first of Stoo Cambridge's A4000s had gone…
At the beginning of November, I started working for wolfSSL. Now that I'm a week…
In my previous post, I had the Amiga 4000 known as Jools mostly repaired, there…
In my last post, I created a list of things I needed to repair on…
View Comments
FWIW, I think line 17 in the last code sample should read:
my_print_func2(format, args);
Thank you for this, I learned something new!
There is definitely a bug there, thanks for pointing it out. Although it shouldn't call `my_print_func2`, that would fail due to a missing parameter and also get in a loop that would blow the stack. It should have been `vprintf()` or any function that takes a `va_list`. I'll fix it now.
Thanks for this artucle Jedi, it truly helped.
I wish you could elaborate on why this justifies a macro.
specifically I'm a bit puzzled regarding this quote of yours:
"This is fine if you own
my_print_func()
and the call chain is only going to work like this. But if, like me, you are in a scenario wheremy_print_func()
andmy_print_func2()
could be called, what can you do? Well, this is where macros come in."X_X
Regardless, thank you for this post!
I don't remember the exact problem I was trying to solve, it was over 4 years ago. But I believe `my_print_func()` was part of a library I was using and couldn't alter it. The library had a way adding your own `printf()` function, and a callback which was `my_print_func2()`. I think I was trying to turn on/off debug/verbose printing in the library and this was the only way around it.