Re: CVS commit: src/usr.sbin/wake

[christos%astron.com@localhost (Christos Zoulas)]

In article <20100110194012.3693A175DA%cvs.netbsd.org@localhost>,
Marc Balmer  <source-changes-d%NetBSD.org@localhost> wrote:
>-=-=-=-=-=-
>
>Module Name:   src
>Committed By:  mbalmer
>Date:          Sun Jan 10 19:40:12 UTC 2010
>
>Modified Files:
>       src/usr.sbin/wake: wake.c
>
>Log Message:
>remove stupid casts.  if lint complains about return values that are ignored,
>lint should fe fixed, and not code being cluttered.

Please explain to me how you propose we fix lint. Or to be more precise,
how does the code convey intent about purposefully ignoring the return
value of a function so that lint can ignore it. FYI this change is against
the current style guide, so if you are going to be pushing for more changes
like that, I suggest that you get consensus about changing the style guide
first. Otherwise I suggest that you revert the change, because it only
serves to produce lint warnings for someone that was linting cleanly before.

christos