Hi Stu
On 03/10/2008, at 12:11 AM, 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.
Did you put that in your PortaWiki?
http://www.flamingspork.com/portawiki/index.php?title=Main_Page
Cheers,
Arjen.
--
Arjen Lentz, Director @ Open Query
Training and Expertise for MySQL in Australia and New Zealand
http://openquery.com.au/training/ (ph. +61-7-3103 0809)
_______________________________________________
Mailing list: https://launchpad.net/~drizzle-discuss
Post to : [email protected]
Unsubscribe : https://launchpad.net/~drizzle-discuss
More help : https://help.launchpad.net/ListHelp
