On Tue, 2012-08-21 at 14:04 +0200, Pavel Březina wrote: > I found this warning while reviewing Michal's transaction patch. > > It was introduced in 9356e8f50436724e985d9a8f5d4f2bc0f5b837bc and it is > visible with the following CFLAGS: > -m64 -mtune=generic -O3 -D_FORTIFY_SOURCE=2 -fstack-protector-all -Wall > -Wextra -Wno-sign-compare -Wno-unused-parameter
Nack. Please don't default to EOK. It makes it hard to keep track of it. Instead, assign it EOK just before the "done:" label. (i.e. "If we got here, we're OK")
signature.asc
Description: This is a digitally signed message part
_______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
