On Mon, 9 Jul 2012, Jordi Ortiz wrote:

Ok as well, but you might try to improve the commit message. You shouldn't try 
to cram everything into the first line - try to make that one as short as 
possible while still describing the commit. If you need to say more than that, 
add extra full sentences below.


Would it be better something like: ?

-----
rtsp: convert rtsp_open_transport_ctx() to non-static.

The RTSP listen mode, which is implemented in rtspdec.c, needs this
functionality and in consequence more visibility for
rtsp_open_transport_ctx() is needed.
-----

You can make it even simpler:

--8<--
rtsp: Make rtsp_open_transport_ctx() non-static

This is required for the upcoming RTSP listen mode.
--8<--

This says all of it without being too wordy, don't you think?

// Martin
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to