About to change about 250 instances, and then there will be no going
back...
Orders from Jay the merciless task master...! ;)
On Oct 2, 2008, at 4:11 PM, Stewart Smith wrote:
On Thu, Oct 02, 2008 at 09:04:47AM -0400, Jay Pipes wrote:
Paul McCullagh wrote:
It's probably useful to go through each of the patches there and
re-do
some of the work. A bunch of the compiler warnings I solved in
rather
silly ways. e.g. unused parameters, i just did (void)foo; instead
of the
better solution of removing the parameter.
I'm OK with this solution for the moment. Some of these cannot be
changed because they involve callbacks which sometimes/may use the
parameters.
-1 Please let's use __attribute__((unused)) as it is clearer and
indicates to the caller that the argument is not currently used...the
(void)foo; approach doesn't make it clear in the function declaration
that the parameter isn't used.
I remember previously having portability issues with this approach
(rather than cast to (void))... can't remember which platform
though...
I'm guessing HPUX or Solaris or Windows... they usually cause me pain
and memory loss.
--
Stewart Smith
--
Paul McCullagh
PrimeBase Technologies
www.primebase.org
www.blobstreaming.org
pbxt.blogspot.com
_______________________________________________
Mailing list: https://launchpad.net/~drizzle-discuss
Post to : [email protected]
Unsubscribe : https://launchpad.net/~drizzle-discuss
More help : https://help.launchpad.net/ListHelp
