vle-forge / vle

Virtual Laboratory Environment
http://www.vle-project.org
GNU General Public License v3.0
21 stars 22 forks source link

About std::ostream* os parameter of RemoteManager::start #348

Closed rtrepos closed 5 years ago

rtrepos commented 6 years ago

Is this really usefull to provide this function signature. The os parameter is used to write information messages but there is already a Context embedded into the RemoteManager that can be used to log messages. Beside, the errors are written with the context from the RemoteManager constructor.

One could remove the os parameter and rely on context->log_fn functor to write information messages.

quesnel commented 6 years ago

Yes. We can delete this useless parameter and move error code to the context functor.

Be careful however. We need to limit API break between VLE 2.0 and 2.1 to restrict package changes.

rtrepos commented 5 years ago

Done in commit 3011028ddcf429261588946bd5b4d65c77b2c753