[Freeipa-devel] [PATCH][SSSD] Clean up warnings in monitor.c
Sumit Bose
sbose at redhat.com
Tue May 12 06:38:17 UTC 2009
Stephen Gallagher schrieb:
> Updated patch, also cleaning up warning in confdb.c
>
> On 05/11/2009 02:43 PM, Stephen Gallagher wrote:
>> GCC 4.4.0 reveals more warnings than previous versions.
>>
>> Clean up function prototype, return without value and pointer signedness
>> warnings.
>>
All changes make sense to me. Wouldn't it be nicer to have
"opt_config_file = talloc_strdup(NULL, CONFDB_DEFAULT_CONFIG_FILE);"
after server_setup? Then you can use main_ctx and do not have to create
a top level context for opt_config_file.
bye,
Sumit
More information about the Freeipa-devel
mailing list