Bridges.h incorrect throw documentation about bridges-cxx HOT 10 CLOSED

I think I have fixed all of them.

from bridges-cxx.

setAssignment(int) is still documented as throwing an exception.
If the validation fails, bridges calls exit(EXIT_FAILURE), but this still doesn't throw.

from bridges-cxx.

I see a throw in the try block (for catching negative assignment numbers. Wouldnt that be caught in the catch block?

from bridges-cxx.

Yeah, exactly. The function throws internally, but also catches it internally, so for documentation purposes it shouldn't be declared as throwing an exception because it will always catch itself, without actually throwing to the user who is calling it.

And again I would recommend changing it like you have for the others. In that, since you already have a check for the throw, it would be better to put the catch calls in the if statement and leave the remainder in the else, without using a try/catch or manual throw altogether.

from bridges-cxx.

removed try/catch in setAssignment

from bridges-cxx.

Looks good, but it is printing the error message to cout.
This should be changed to cerr, to keep in line with similar situations.

from bridges-cxx.

fixed.

from bridges-cxx.

setAssignment still seems to be printing to cout not cerr. Additionally Bridges.h's server error msg and visualize functions are also printing to cout instead of cerr.

from bridges-cxx.

fixed.

from bridges-cxx.

visualize is still using cerr

Also since this is an if else list, I would exclude the explicit "None" check and just leave its code in an else statement at the end, that way if for some odd reason the visualizer type is not one of these types it will still be handled.

from bridges-cxx.