Review Request 312: Remove extra semi-colons

reviewboard at gallagherhome.com reviewboard at gallagherhome.com
Thu May 9 09:44:06 UTC 2013



> On May 9, 2013, 8:17 a.m., Radek Novacek wrote:
> > Ship It!

I don't have git access, thanks for pushing.


- stefw


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/312/#review354
-----------------------------------------------------------


On May 8, 2013, 1:45 p.m., stefw wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard-openlmi.rhcloud.com/r/312/
> -----------------------------------------------------------
> 
> (Updated May 8, 2013, 1:45 p.m.)
> 
> 
> Review request for OpenLMI Developers.
> 
> 
> Repository: openlmi-providers
> 
> 
> Description
> -------
> 
> 15:08:34 **** Build of configuration Default for project openlmi-providers ****
> make all 
> [ 11%] Built target openlmicommon
> [ 22%] Building C object src/realmd/CMakeFiles/cmpiLMI_Realmd.dir/LMI_HostedRealmdServiceProvider.c.o
> In file included from /data/projects/openlmi-providers/src/realmd/LMI_HostedRealmdServiceProvider.c:5:0:
> /data/projects/openlmi-providers/src/globals.h:33:0: warning: "ORGID" redefined [enabled by default]
>  #define ORGID "LMI"
>  ^
> In file included from /data/projects/openlmi-providers/src/realmd/LMI_HostedRealmdServiceProvider.c:4:0:
> /data/projects/openlmi-providers/src/realmd/rdcp_util.h:12:0: note: this is the location of the previous definition
>  #define ORGID "LMI_Realmd"
>  ^
> /data/projects/openlmi-providers/src/realmd/LMI_HostedRealmdServiceProvider.c:233:28: warning: ISO C does not allow extra ‘;’ outside of a function [-Wpedantic]
>      "instance association");
>                             ^
> [ 33%] Building C object src/realmd/CMakeFiles/cmpiLMI_Realmd.dir/LMI_RealmdKerberosRealmProvider.c.o
> In file included from /data/projects/openlmi-providers/src/realmd/LMI_RealmdKerberosRealmProvider.c:6:0:
> /data/projects/openlmi-providers/src/realmd/rdcp_util.h:12:0: warning: "ORGID" redefined [enabled by default]
>  #define ORGID "LMI_Realmd"
>  ^
> In file included from /data/projects/openlmi-providers/src/realmd/LMI_RealmdKerberosRealmProvider.c:3:0:
> /data/projects/openlmi-providers/src/globals.h:33:0: note: this is the location of the previous definition
>  #define ORGID "LMI"
>  ^
> /data/projects/openlmi-providers/src/realmd/LMI_RealmdKerberosRealmProvider.c:627:23: warning: ISO C does not allow extra ‘;’ outside of a function [-Wpedantic]
>      "instance method");
>                        ^
> [ 44%] Building C object src/realmd/CMakeFiles/cmpiLMI_Realmd.dir/LMI_RealmdRealmProvider.c.o
> In file included from /data/projects/openlmi-providers/src/realmd/LMI_RealmdRealmProvider.c:7:0:
> /data/projects/openlmi-providers/src/realmd/rdcp_util.h:12:0: warning: "ORGID" redefined [enabled by default]
>  #define ORGID "LMI_Realmd"
>  ^
> In file included from /data/projects/openlmi-providers/src/realmd/LMI_RealmdRealmProvider.c:4:0:
> /data/projects/openlmi-providers/src/globals.h:33:0: note: this is the location of the previous definition
>  #define ORGID "LMI"
>  ^
> /data/projects/openlmi-providers/src/realmd/LMI_RealmdRealmProvider.c:326:23: warning: ISO C does not allow extra ‘;’ outside of a function [-Wpedantic]
>      "instance method");
>                        ^
> [ 55%] Building C object src/realmd/CMakeFiles/cmpiLMI_Realmd.dir/LMI_RealmdServiceProvider.c.o
> In file included from /data/projects/openlmi-providers/src/realmd/LMI_RealmdServiceProvider.c:6:0:
> /data/projects/openlmi-providers/src/realmd/rdcp_util.h:12:0: warning: "ORGID" redefined [enabled by default]
>  #define ORGID "LMI_Realmd"
>  ^
> In file included from /data/projects/openlmi-providers/src/realmd/LMI_RealmdServiceProvider.c:3:0:
> /data/projects/openlmi-providers/src/globals.h:33:0: note: this is the location of the previous definition
>  #define ORGID "LMI"
>  ^
> /data/projects/openlmi-providers/src/realmd/LMI_RealmdServiceProvider.c:631:23: warning: ISO C does not allow extra ‘;’ outside of a function [-Wpedantic]
>      "instance method");
>                        ^
> [ 66%] Building C object src/realmd/CMakeFiles/cmpiLMI_Realmd.dir/LMI_ServiceAffectsRealmdRealmProvider.c.o
> In file included from /data/projects/openlmi-providers/src/realmd/LMI_ServiceAffectsRealmdRealmProvider.c:4:0:
> /data/projects/openlmi-providers/src/globals.h:33:0: warning: "ORGID" redefined [enabled by default]
>  #define ORGID "LMI"
>  ^
> In file included from /data/projects/openlmi-providers/src/realmd/LMI_ServiceAffectsRealmdRealmProvider.c:3:0:
> /data/projects/openlmi-providers/src/realmd/rdcp_util.h:12:0: note: this is the location of the previous definition
>  #define ORGID "LMI_Realmd"
>  ^
> /data/projects/openlmi-providers/src/realmd/LMI_ServiceAffectsRealmdRealmProvider.c:252:28: warning: ISO C does not allow extra ‘;’ outside of a function [-Wpedantic]
>      "instance association");
>                             ^
> [ 77%] Building C object src/realmd/CMakeFiles/cmpiLMI_Realmd.dir/rdcp_dbus.c.o
> /data/projects/openlmi-providers/src/realmd/rdcp_dbus.c:697:1: warning: ‘dbus_method_append_args_tuple’ defined but not used [-Wunused-function]
>  dbus_method_append_args_tuple(DBusMessage  *message, GVariant *args, GError **g_error)
>  ^
> /data/projects/openlmi-providers/src/realmd/rdcp_dbus.c:777:1: warning: ‘marshal_dbus_dict_string_entry’ defined but not used [-Wunused-function]
>  marshal_dbus_dict_string_entry(DBusMessageIter *array, const char *name, const char *value, GError **g_error)
>  ^
> [ 88%] Building C object src/realmd/CMakeFiles/cmpiLMI_Realmd.dir/rdcp_error.c.o
> [100%] Building C object src/realmd/CMakeFiles/cmpiLMI_Realmd.dir/rdcp_util.c.o
> Linking C shared library libcmpiLMI_Realmd.so
> Generating .reg file from library for Realmd
> [100%] Built target cmpiLMI_Realmd
> 
> 15:08:37 Build Finished (took 3s.194ms)
> 
> 
> Diffs
> -----
> 
>   src/realmd/LMI_HostedRealmdServiceProvider.c 08732c1e9deac41df4fcda2d1088107f7d9be10c 
>   src/realmd/LMI_RealmdKerberosRealmProvider.c 5dc90d2b5faefe264c652d054d4770d57ee02fe4 
>   src/realmd/LMI_RealmdRealmProvider.c d8e0a188f0f822004c4f082fb52730c61479bef1 
>   src/realmd/LMI_RealmdServiceProvider.c 01fc0b839d0cc7267ef623bba9aad14581fc585d 
>   src/realmd/LMI_ServiceAffectsRealmdRealmProvider.c 70d65f4738ad4aa27d6bdcf8bb73ed465bc2c246 
> 
> Diff: http://reviewboard-openlmi.rhcloud.com/r/312/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> stefw
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.fedorahosted.org/pipermail/openlmi-devel/attachments/20130509/4a51220b/attachment-0001.html>


More information about the openlmi-devel mailing list