[Osi] warning on comment in OsiSolverInterface.hpp (trunk)

[Matthew Saltzman]

On Fri, 2011-01-07 at 14:18 -0800, Lou Hafer wrote: 
> Matt,
> 
> > [ What's the point of ...
> 
>   virtual bool setHintParam(OsiHintParam key, bool yesNo=true,
>                             OsiHintStrength strength=OsiHintTry,
>                             void */*otherInformation*/ = NULL) {
> > ... ]
> 
> 	Yeah, this puzzled me, too, the first time I ran across it.  My
> interpretation is that it's one way of clearly indicating that the parameter is
> not used, while still satisfying the requirements of the method signature.
> 
> 	I'm happy to get rid of it as I find it, particularly if it's causing
> warnings.  Is there anyone reading who's willing to speak in favour of keeping
> this idiom?

I can take it or leave it, I guess.  But to get rid of the warning
without losing the meta-information (such as it is), replace with

        void * /*otherInformation*/ = NULL

(Note the additional space.)

Matt

> 
> 							Lou
> 
> _______________________________________________
> Osi mailing list
> Osi at list.coin-or.org
> http://list.coin-or.org/mailman/listinfo/osi

-- 
                Matthew Saltzman

Clemson University Math Sciences
mjs AT clemson DOT edu
http://www.math.clemson.edu/~mjs