This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/1934/ |
src/software-dbus/sw-utils.c (Diff revision 1) | |||
---|---|---|---|
void get_pk_repo(const char *repo_id_p, PkRepoDetail **repo_p, char *error_msg, |
|||
1004 | if (!enabled) |
||
1005 | continue; |
Missing possiblity of listing disabled repos.
src/software-dbus/sw-utils.c (Diff revision 1) | |||
---|---|---|---|
void get_pk_repo(const char *repo_id_p, PkRepoDetail **repo_p, char *error_msg, |
|||
1011 | g_ptr_array_ref(array); |
||
1012 | *garray = array; |
Why 2 lines instead of 1? This looks to me less intuitive.
- Peter Schiffer
On July 25th, 2014, 2:44 p.m. CEST, Michal Minar wrote:
Review request for OpenLMI Developers.
By Michal Minar.
Updated July 25, 2014, 2:44 p.m.
Repository:
openlmi-providers
Description
Diffs
|