On Wed, 2016-02-10 at 22:22 +0200, Dmitry Kasatkin wrote:
> On Wed, Feb 3, 2016 at 9:06 PM, Mimi Zohar <zohar at linux.vnet.ibm.com> wrote:
> > Add support for measuring and appraising the IMA policy itself.
> >
> > Signed-off-by: Mimi Zohar <zohar at linux.vnet.ibm.com>
>
> Acked-by: Dmitry Kasatkin <dmitry.kasatkin at huawei.com>
>
> But from Documentation/CodingStyle
>
> if (condition)
> do_this();
> else
> do_that();
>
> This does not apply if only one branch of a conditional statement is a single
> statement; in the latter case use braces in both branches:
>
> if (condition) {
> do_this();
> do_that();
> } else {
> otherwise();
> }
>
>
> You have similar issue in other patches as well...
>
> Dmitry
Ok, I'll find and fix them. Thank you for the review!
Mimi
