On Sun, Jan 24, 2016 at 5:28 AM, Timothy Gu <timothyg...@gmail.com> wrote:
> Hi,
>
> On Fri, Jan 22, 2016 at 11:24:19PM +0530, Ganesh Ajjanagadde wrote:
>> +@item h
>> +Dump packets/hex; press to cycle through the 3 states.
>
> What are the three states? Packets, hex, emacs-emulation?

No idea, will need to inspect the code closely. This was copied over
from the help text in ffmpeg.c.

>
>> +@item d
>> +Accept subsequent entry up to carriage return as debug mode.
>
> What is "subsequent entry"? "abcd\r"?

Suspect so, but note that precise specification depends on the
semantics of sscanf. I think "subsequent entry as integer" is a
reasonable approximation for that particular problem. Other problems
with this may remain.

>
> Please send the new patch for review.

Indeed, felt the same way. Will have to wait till I get back; or if
someone is interested, can be completed now. Thanks.

>
> Timothy
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to