Check whether the current approach of storing this information in the class hierarchy of the externally visible components is the right way to go, or whether this information should be held local in the internal object.

Member visr::impl::ComponentImplementation::period () const

: Check whether this should be made inline again (adding the dependency to the runtime container (aka SignalFlow).

Member visr::impl::ComponentImplementation::registerParameterPort (ParameterPortBaseImplementation *port)

consider making this a template method to share the implementation between audio and parameter ports.

Member visr::impl::ComponentImplementation::unregisterParameterPort (ParameterPortBaseImplementation *port)

consider making this a template method to share the implementation between audio and parameter ports.

Class visr::impl::ParameterConnection

This should move into CompositeComponent.

Member visr::impl::PortBaseImplementation::~PortBaseImplementation ()

Check whether this still holds.

Member visr::objectmodel::Object::cInvalidChannelIndex

Decide whether this type must be visible on the outside.

Class visr::objectmodel::ObjectParser

revise class hierarchy (does it make sense to let the base of the parser object hierarchy to perform actual work, or should there be an abstract interface class on top of it?

Member visr::objectmodel::PointSourceParser::parsePosition (boost::property_tree::ptree const &posTree, Object::Coordinate &x, Object::Coordinate &y, Object::Coordinate &z)

Consider making this a more general facility.

Member visr::objectmodel::PointSourceWithReverb::cNumDiscreteReflectionBiquads

Consider making this a systeem-wide configuration option.

Member visr::ParameterPortBase::setProtocol (CommunicationProtocolBase *protocol)=0

Reconsider interface.

Member visr::pml::ListenerPosition::set (Coordinate newX, Coordinate newY, Coordinate newZ=0.0f)

rename to setPosition()

Member visr::rbbl::CrossfadingConvolverUniform< SampleType >::transformImpulseResponse (SampleType const *ir, std::size_t irLength, FrequencyDomainType *result, std::size_t alignment=0) const

The semantics of the alignment parameter would be clearer if it would be in bytes.

Member visr::rbbl::InterpolatingConvolverUniform< SampleType >::setInterpolants (InterpolationParameterSet const &param)

Decide whether to clear the existing interpolants.

Class visr::rcl::DelayMatrix

Implement flexible transition length (at the moment, the block length is always used

general cleanup (including more comprehensive range checks for the delay)

Class visr::rcl::DelayVector

Implement flexible transition length (at the moment, the block length is always used

general cleanup (including more comprehensive range checks for the delay)

Member visr::rcl::GainMatrix::setup (std::size_t numberOfInputs, std::size_t numberOfOutputs, std::size_t interpolationSteps, SampleType initialGain=static_cast< SampleType >(0.0), bool controlInput=true)

Describe the complete semantics of the transition.

Member visr::rcl::PanningCalculator::CoefficientType

maybe this should become a template parameter.

Member visr::rcl::PositionDecoder::PositionDecoder (SignalFlowContext const &context, char const *name, CompositeComponent *parent, panning::XYZ const &offsetKinect, float qw=1.0f, float qx=0.0f, float qy=0.0f, float qz=0.0f)

Provide a general implementation for transforming the coordinate system.

Member visr::rcl::SceneEncoder::process () override

Create a triggering/timing method to control the output rate.

Member visr::rcl::SparseGainMatrix::SparseGainMatrix (SignalFlowContext const &context, char const *name, CompositeComponent *parent, std::size_t numberOfInputs, std::size_t numberOfOutputs, std::size_t interpolationSteps, std::size_t maxRoutingPoints, rbbl::SparseGainRoutingList const &initialRoutings=rbbl::SparseGainRoutingList(), ControlPortConfig controlInputs=ControlPortConfig::No)

Describe the complete semantics of the transition.

Member visr::reverbobject::LateReverbFilterCalculator::LateReverbFilterCalculator (SignalFlowContext const &context, char const *name, CompositeComponent *parent, std::size_t numberOfObjects, SampleType lateReflectionLengthSeconds, std::size_t numLateReflectionSubBandLevels, std::size_t maxUpdatesPerPeriod=0)

Consider making lateReflectionSubbandLevels optional.

Member visr::rrl::AudioSignalFlow::process (SampleType const *captureSamples, std::size_t captureChannelStride, std::size_t captureSampleStride, SampleType *playbackSamples, std::size_t playbackChannelStride, std::size_t playbackSampleStride)

Provide a callback function for interfacing with audio interfaces.

Member visr::rrl::AudioSignalFlow::processFunction (void *userData, SampleType const *const *captureSamples, SampleType *const *playbackSamples, bool &status)

After the redesign, the translation to a callback function (and discarding the object pointer) needs to be done somewhere else!

Member visr::rrl::isPlaceholderPort (impl::PortBaseImplementation const *const port)

consider moving to a graph checking and manipulation library to be defined.

Class visr::rrl::ParameterConnectionMap

Public intheritance from std::map is a short-term solution to permit its forward in order make this a private header (not exported to other projects).

Remove the use of this class from the public interface of rrl::AudioSignalFlow.