On Tue, Jan 29, 2013 at 09:27:22AM -0000, Dmitrijs Ledkovs wrote:
> > get_subdir (const char *dir, const char *suffix, int create)
> > {
> > [...]
> > nih_assert (dir != NULL);
> > [...]
> > if (dir && dir[0] == '/') {
> > [...]
> > Looks like a redundant check for dir's nullness.
> It's a check for absolute path.
That's written 'if (dir[0] == '/')'. The check for 'dir &&' is what's
redundant.
--
https://code.launchpad.net/~jamesodhunt/upstart/add-session-file/+merge/144881
Your team Upstart Reviewers is subscribed to branch lp:upstart.
--
upstart-devel mailing list
[email protected]
Modify settings or unsubscribe at:
https://lists.ubuntu.com/mailman/listinfo/upstart-devel
