Review Request 313: Remove unused functions from realmd provider

reviewboard at gallagherhome.com reviewboard at gallagherhome.com
Thu May 9 09:43:30 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/313/#review355
-----------------------------------------------------------


On May 8, 2013, 1:47 p.m., stefw wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard-openlmi.rhcloud.com/r/313/
> -----------------------------------------------------------
> 
> (Updated May 8, 2013, 1:47 p.m.)
> 
> 
> Review request for OpenLMI Developers.
> 
> 
> Repository: openlmi-providers
> 
> 
> Description
> -------
> 
> Fix warnings:
> 
> 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/rdcp_dbus.c b05e5c981340c09ec09ecaaaf0b8583db9696e63 
> 
> Diff: http://reviewboard-openlmi.rhcloud.com/r/313/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> stefw
> 
>

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


More information about the openlmi-devel mailing list