Attention is currently required from: jolly.

Patch set 2:Code-Review +1

View Change

1 comment:

• File src/ipa/main.c:

  • Patch Set #2, Line 151: static struct ipa_context *ctx = NULL;

    I would suggest to move this to the top of the module and to add a comment.

    (I don't know if this is a coding style thing, but I always prefer to declare variables at the beginning of a module or function, it is then much easier to see what states a module or function has.)

To view, visit change 43653. To unsubscribe, or for help writing mail filters, visit settings.