[lvm-devel] Allow use headers in C++.
Milan Broz
mbroz at redhat.com
Tue Jun 15 11:50:11 UTC 2010
On 06/15/2010 01:43 PM, Mike Snitzer wrote:
> On Mon, Jun 14 2010 at 12:20pm -0400,
> Milan Broz <mbroz at redhat.com> wrote:
>
>> Remove C++ private keyword from header.
> ...
>> diff --git a/libdm/libdm-report.c b/libdm/libdm-report.c
>> index e13115a..2b044b5 100644
>> --- a/libdm/libdm-report.c
>> +++ b/libdm/libdm-report.c
>> @@ -566,7 +566,7 @@ struct dm_report *dm_report_init(uint32_t *report_types,
>> const char *output_separator,
>> uint32_t output_flags,
>> const char *sort_keys,
>> - void *private)
>> + void *private_data)
>> {
>> struct dm_report *rh;
>> const struct dm_report_object_type *type;
>> @@ -588,7 +588,7 @@ struct dm_report *dm_report_init(uint32_t *report_types,
>> rh->separator = output_separator;
>> rh->fields = fields;
>> rh->types = types;
>> - rh->private = private;
>> + rh->private = private_data;
>>
>> rh->flags |= output_flags & DM_REPORT_OUTPUT_MASK;
>
> Should struct dm_report's "private" be renamed too?
Not needed, that's not part of external API/ABI but internal to library.
(There are more such conflicting names, but these are not problem in compilation,
because this is always compiled in C only.)
Milan
More information about the lvm-devel
mailing list