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. -jay _______________________________________________ Mailing list: https://launchpad.net/~drizzle-discuss Post to : [email protected] Unsubscribe : https://launchpad.net/~drizzle-discuss More help : https://help.launchpad.net/ListHelp
