Il 09/10/2012 16:58, Luiz Capitulino ha scritto: >> > + error_set_errno(errp, errno, QERR_SOCKET_CONNECT_FAILED); > The patch look fine, but as I said in my previous email I really dislike > seeing QERR_ macros usage in new code. If the problem here is to duplicate > the error message, then maybe we could put this connect() block in a wrapper.
Again: one thing at a time. The only obvious step is to remove QERR_ constants that are used just once. Everything else should be done carefully because if later you decide to add something to the errors (for example the bind or connect argument) you have no protection against typos, etc. Adding new QERR_ constants is somewhat harmful, but using the old ones absolutely is not. Paolo