Project

General

Profile

Idea #968

reconfigure interface

Added by Mark Messier over 8 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
High
Category:
Infrastructure
Target version:
Start date:
02/14/2011
Due date:
% Done:

100%

Estimated time:
Experiment:
-
SSI Package:
Duration:

Description

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)?

Associated revisions

Revision 223a754e (diff)
Added by Christopher Green over 8 years ago

Simplify interface of reconfigure per issue #968. Will need adjustments to NOvA code on release.

History

#1 Updated by Marc Paterno over 8 years ago

  • Category set to Infrastructure
  • Assignee set to Christopher Green
  • Priority changed from Normal to High
  • Target version set to 0.06.00

#2 Updated by Christopher Green over 8 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.

#3 Updated by Christopher Green over 8 years ago

  • Status changed from Resolved to Closed


Also available in: Atom PDF