I have some more question. Why is the documentation in the implementation file? I was thinking it is in the header file. How can I produce a documentation with doxygen? There (more...)

Hi Peter, I'd personally stick to the following style: //! Short description. /*! \param first First parameter description. \param second Second parameter description. \param Nth Nth parameter description. \returns true if something, false otherwise. Description (if the (more...)

Hi Markus, I have some more question. Why is the documentation in the implementation file? I was thinking it is in the header file. How can I produce a documentation with (more...)

Hi Petr, I updated the docs in TcpSocketBuffer to use \param. The Doxygen output looks really nice! //! Writes \a bytes to the write buffer. /*! \param bytes are the bytes that should be (more...)

Hi Peter, I have some more question. Why is the documentation in the implementation file? I was thinking it is in the header file. How can I produce a documentation with (more...)

Hi Petr, I just noted I have forgotten to comment on this: - I’d move SocketTransport class members initialization into the initialization part of the constructor. - I’d add const modifiers to canRead(), (more...)

Hi Petr, I’ve made several changes and pushed them to http://bzr.calitko.org/developers/s3rvac/calitko-peter so you can have a look and possibly merge some changes with your branch: Great! I'll look at it and merge (more...)

Hi Peter, thanks for that tip with creating a new branch on http://bzr.calitko.org/developers/s3rvac so I can easily express and public my changes in your code! I've made several changes and pushed (more...)

Hi Petr, Yes that is a good remark! I agree it is so much better to fix all warnings as they are potential bugs! I don't think any of them have (more...)

Just saw that the code snippet got messed up and additionally I found a mistake. auto_ptr <PacketBase> tempPacket (copy()); if (tempPacket->doParse (rawHeader, rawPayload, rawTrailer)) { *this = *tempPacket; (more...)