[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Bug-xorriso] Link errors against libisoburn
From: |
Thomas Schmitt |
Subject: |
Re: [Bug-xorriso] Link errors against libisoburn |
Date: |
Tue, 15 Mar 2016 18:20:42 +0100 |
Hi,
Andrei Borzenkov wrote:
> Actually you probably need to wrap the whole include file into
>
> #ifdef __cplusplus
> extern "C" {
> #endif
>
> ...
>
> #ifdef __cplusplus
> }
> #endif
This is indeed a good proposal for all four public headers.
Actually libburn.h seems to have such a provision:
--------------------------------------------------------------------------
#ifndef DOXYGEN
#if defined(__cplusplus)
#define BURN_BEGIN_DECLS \
namespace burn { \
extern "C" {
#define BURN_END_DECLS \
} \
}
#else
#define BURN_BEGIN_DECLS
#define BURN_END_DECLS
#endif
BURN_BEGIN_DECLS
#endif
... all API declarations ...
#ifndef DOXYGEN
BURN_END_DECLS
#endif
--------------------------------------------------------------------------
In libburnia SVN this is blamed on revision 1. I.e. Mario Danic's initial
import of libburn-0.2 when we forked libburnia from icculus.org/libburn
back in 2006.
I am not happy with carrying on Doxygen preparations. But probably it is
best to copy this gesture to the other three headers.
Now i have to find out what "namespace burn {" shall mean ...
Have a nice day :)
Thomas