On 11/28/11 11:28 AM, Diego Biurrun wrote:
On Tue, Nov 08, 2011 at 02:31:45PM +0100, Diego Biurrun wrote:
On Sun, Nov 06, 2011 at 03:30:51PM +0100, Reinhard Tartler wrote:
On Sun, Oct 30, 2011 at 10:08 PM, Diego Biurrun<[email protected]>  wrote:

--- a/cmdutils.h
+++ b/cmdutils.h
@@ -190,7 +190,7 @@ void parse_loglevel(int argc, char **argv, const OptionDef 
*options);
  *
  * @param s  Corresponding format context.
  * @param st Stream from s to be checked.
- * @param spec A stream specifier of the [v|a|s|d]:[<stream index>] form.
+ * @param spec A stream specifier of the [v|a|s|d]:[\<stream index\>] form.
  *

Why that change?
http://libav.org/doxygen/master/cmdutils_8h.html#a632b71e4b56c5ee8556a407bbecbcef8
doesn't look broken to me at all.

Without the escaping we get the following warning:

libav/cmdutils.h:190: warning: Unsupported xml/html tag<stream>  found

Besides, while that documentation explains the format of the string to
be passed, it still leaves me the following questions:

  - what's the meaning of the v, a, s and d prefixes?
  - what happens if the string couldn't be parsed (I /guess/ the
function returns some negative value)

Patches welcome :)

... ping ...


The patch seems fine as is, we can improve later.

lu

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to