hal: Branch 'master' - 3 commits
Danny Kukawka
danny.kukawka at web.de
Mon Mar 3 14:42:11 PST 2008
On Montag, 3. März 2008, David Zeuthen wrote:
> On Mon, 2008-03-03 at 23:23 +0100, Danny Kukawka wrote:
> > - hal_set_proc_title ("hald-addon-storage: no polling because %s is
> > locked via HAL and O_EXCL", device_file); + hal_set_proc_title
> > ("hald-addon-storage: no polling because %s locked via HAL and O_EXCL",
> > device_file);
>
> I'm not a native speaker but I think removing the "is" is (pun!) wrong.
I'm also not a native speaker but it would fit with this length better in 'ps
aux' output. I'm currently only afraid, that it get to long for ps output and
with that unreadable e.g. on 1024x768 or smaller displays.
> Also, I'm not a big fan of functions returning in the middle; "goto"
> error handling is much more elegant. So please don't change that.
> Anyway, I'm sure a lot of people feel different about this.
I can live with that, also if this would save us one variable we don't need.
Danny
More information about the hal
mailing list