[Cbc] unnecessary complicated code in CbcOrClpParam.cpp?

[John Forrest]

Tobias,

I am sure you can come up with worse examples of my bad coding - fixed 
in stable and trunk.

John Forrest

On 17/08/16 09:04, Tobias Stengel wrote:
>
> Hello,
>
> I found
>
> field =argv[CbcOrClpRead_mode++];
>
> // -- means import from stdin
>
> field ="-";
>
> in CbcOrClpParam.cpp, line 1411-1413. As far as I can see,
>
> ++CbcOrClpRead_mode;
>
> // -- means import from stdin
>
> field ="-";
>
> would do the same. As nobody has submitted a pull request on github: 
> what ist the prefered way for contributing such small changes? Posting 
> on the mailing list and waiting for Forrest looks complicated.
>
> Tobias
>
>
>
> _______________________________________________
> Cbc mailing list
> Cbc at list.coin-or.org
> http://list.coin-or.org/mailman/listinfo/cbc


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://list.coin-or.org/pipermail/cbc/attachments/20160817/2b4a8d35/attachment.html>