Code review comment for lp:~ballogy/bamf/fix-current-desktop

Revision history for this message
Marco Trevisan (TreviƱo) (3v1n0) wrote :

Thanks for your patch.

27 - if (!exec)
28 + if (!exec || exec[0] == '\0')

In the case that (exec[0] == '\0') you also need to free it.

Overall that's fine. Would you mind also to add a small test case into test-matcher.c?

« Back to merge proposal