The current reconfigure() interface for modules and services take an istream and an ostream as its first two arguments and a ParameterSet as its third. In practice, I'm finding that the istream and ostream are almost always unused in our code and users must invent some way of providing dummy streams for these arguments. Could we reorder the arguments so that the istream and ostream could be made optional: reconfigure(ParameterSet&p, istream& in=std::cin, ostream& out=std::cout)?
#2 Updated by Christopher Green over 9 years ago
- Status changed from New to Resolved
- % Done changed from 0 to 100
reconfigure() now takes one argument only: the parameter set. See commit 223a754.
If this is not what was wanted, please let me know and we can adjust. Note that NOvA code will need to be changed to coincide with the release of this code.