On 09/01/2010 03:51 PM, Rich Megginson wrote:
"backend: %s \n", be->be_name, isglobal?"(global mode)":"");
"backend: %s%s%s\n", be->be_name, isglobal?" ":"", isglobal?"(global mode)":"");
Is there any reason that this section shouldn't just use 2 strings in the format statement? Something like this:
"backend: %s%s\n", be->be_name, isglobal?" (global mode)":"");
Other than that minor nitpick, the changes look good.
-NGK
Nathan Kinder wrote:
On 09/01/2010 03:51 PM, Rich Megginson wrote:
"backend: %s \n", be->be_name, isglobal?"(global
mode)":"");
"backend: %s%s%s\n", be->be_name, isglobal?"
":"", isglobal?"(global mode)":"");
Is there any reason that this section shouldn't just use 2 strings in the format statement? Something like this:
"backend: %s%s\n", be->be_name, isglobal?" (global mode)":"");
Other than that minor nitpick, the changes look good.
Fixed that. Thanks!
Pushed to master.
-NGK
389-devel@lists.fedoraproject.org