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 -- 389-devel mailing list 389-devel@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/389-devel