reSIProcate/stack  9694
Todo List
Member resip::BranchParameter::setSigcompCompartment (const Data &)
The encoding here could be more efficient.
Class resip::ConnectionBase
check id connectionId makes sense here
Member resip::ConnectionBase::transport () const
should be reference
Member resip::Contents::addBuffer (char *buf)
- is this being used? -- is the buffer list being used as a list?
Member resip::Contents::clear ()
!bwc! Calls freeMem(), then reverts members to a default state (including setting pointers to 0)
Member resip::Contents::Contents (const Contents &rhs, HeaderFieldValue::CopyPaddingEnum e)
internal documentation
Member resip::Contents::Contents (const HeaderFieldValue &headerFieldValue, HeaderFieldValue::CopyPaddingEnum e, const Mime &contentsType)
internal documentation
Member resip::Contents::getContents (const Mime &)
- unimplemented - decide fate
Class resip::ExpiresCategory
Seems like this and UInt32Category (numeric _with_ a comment) could be renamed to something a little more self-explanatory.
Class resip::MessageWaitingContents::AccountHeader
This is an awkward attempt to make accessing the MessageAccount Uri operate like accessing the other MessageWaiting headers even though the user of these calls has to treat them differently to being with since the two header calls return different classes. ugh. .mjf.
Class resip::ParserCategory

Maybe a better name? IHaveParams? ElemWithParams?

Member resip::ParserContainer< T >::reverse () const
!bwc! optimize this (we are copying each ParserContainer twice)
Member resip::ParserContainerBase::encode (const Data &headerName, EncodeStream &str) const
add support for headers that are allowed to be empty like Supported, Accept-Encoding, Allow-Events, Allow, Accept, Accept-Language
Member resip::SipMessage::clone () const
.dlb. sure would be nice to have overloaded return value here..
Member resip::SipMessage::defineHeader (RetryAfter,"Retry-After", UInt32Category,"RFC 3261")
!dlb! this one is not quite right -- can have (comment) after field value
Member resip::SipMessage::SipMessage (const SipMessage &message)
.dlb. public, allows pass by value to compile.
Member resip::SipStack::setContentLengthChecking (bool check)
should this be fixed to work with other applicable transports? []
Member resip::SipStack::statisticsManagerEnabled ()
is this documented correctly? [!]
Class resip::TimerQueue< T >

!dcm! - refactor, templatize

.dlb. timer wheel for transaction-bound timers and a heap for everything longer.

Member resip::TransactionUser::addDomain (const Data &domain)
Make this case-insensitive.
Member resip::Transport::send (std::auto_ptr< SendData > data)=0

!bwc! What we do with a SendData is flexible.

!bch! Should this be protected and not public? !bwc! TransportSelector uses this directly for retransmissions.

Member resip::Transport::shutdown ()
?dcm? pure virtual protected method to enforce this?
Member resip::Transport::startOwnProcessing ()=0
?dcm? avoid the received a message but haven't added a transport to the TransportSelector race, but this might not be necessary.
Member resip::Transport::Transport (Fifo< TransactionMessage > &rxFifo, int portNum, IpVersion version, const Data &interfaceObj, const Data &tlsDomain=Data::Empty, AfterSocketCreationFuncPtr socketFunc=0, Compression &compression=Compression::Disabled, unsigned transportFlags=0)
Note that because of InternalTransport's constructor, tlsDomain is always set to Data::Empty at construction time, in practice.
Member resip::Tuple::isV4 () const
!dcm! -- should deprecate asap