Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On 01-06-2018 01:16 PM, Gyan Doshi wrote: I'm fairly confident it will since @subheading also works online for developer.html, but not locally. However, this shouldn't be blocking your patch since it is an independent issue. Ok. Will push in a day or so. Thanks, Gyan Revised and pushed.

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On 31-05-2018 11:57 PM, Lou Logan wrote: On Wed, May 30, 2018, at 9:56 PM, Gyan Doshi wrote: Can you run this on the server to check if subsubheadings are rendered? I'm fairly confident it will since @subheading also works online for developer.html, but not locally. However, this

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On Wed, May 30, 2018, at 9:56 PM, Gyan Doshi wrote: > > Can you run this on the server to check if subsubheadings are rendered? I'm fairly confident it will since @subheading also works online for developer.html, but not locally. However, this shouldn't be blocking your patch since it is an

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On 31-05-2018 02:00 AM, Lou Logan wrote: On Tue, May 29, 2018, at 9:43 PM, Gyan Doshi wrote: Works when "--init-file=doc/t2h.pm" is omitted. Didn't investigate further. make uses the init file here. makeinfo is texi2any 6.3 makeinfo --html -I doc --no-split -D config-not-all

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On 31-05-2018 04:35 AM, Michael Niedermayer wrote: If a user consults the relevant parts of the documentation before execution, they should be able to predict how the program will behave. If they do it afterwards, they should understand what the program did. Even though FFmpeg is an open

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On Wed, May 30, 2018 at 10:59:22AM +0530, Gyan Doshi wrote: > > > On 30-05-2018 04:57 AM, Carl Eugen Hoyos wrote: > >2018-05-27 6:16 GMT+02:00, Gyan Doshi : > > > >>v2 attached. > > > >>+In the absence of any map options for a particular output file, ffmpeg > >>inspects the output > >>+format

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On Tue, May 29, 2018, at 9:43 PM, Gyan Doshi wrote: > > They do here. Which version of makeinfo do you have? 6.5 Works when "--init-file=doc/t2h.pm" is omitted. Didn't investigate further. > More importantly, which version is on the project server? 5.2

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On 30-05-2018 04:39 AM, Lou Logan wrote: The subject nit still exists. Fixed locally. Using an itemized list here will look better in my opinion. @itemize @item for video, it is the stream with the highest resolution, @item ... @end itemize Changed. +@subsubheading Example: automatic

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On 30-05-2018 04:57 AM, Carl Eugen Hoyos wrote: 2018-05-27 6:16 GMT+02:00, Gyan Doshi : v2 attached. +In the absence of any map options for a particular output file, ffmpeg inspects the output +format to check which type of streams can be included in it, viz. video, audio and/or

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

2018-05-27 6:16 GMT+02:00, Gyan Doshi : > v2 attached. > +In the absence of any map options for a particular output file, ffmpeg > inspects the output > +format to check which type of streams can be included in it, viz. video, > audio and/or Sorry, what is "viz."? > +subtitles. For each

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On Sun, 27 May 2018 09:46:46 +0530 Gyan Doshi wrote: > I was talking about the vertical margins appled to the 'code' element. That's ugly too. I removed the margin for the code element. > From 60ed76348e70f1b0a25eadde8d886d47be3fca69 Mon Sep 17 00:00:00 2001 > From: Gyan Doshi > Date: Thu, 24

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On 27-05-2018 07:59 PM, Moritz Barsnick wrote: This is incorrect, as it already was before. It doesn't disable *automatic* stream selection, it rather disables mapping any of these stream types at all, *even* if explictly mapped. (IOW: "-vn -map 0:v" never gets you any video stream in the

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On Sun, May 27, 2018 at 09:46:46 +0530, Gyan Doshi wrote: > +The @code{-vn}, @code{-an}, @code{-sn} options can be used to skip automatic > stream selection > +for video, audio, and subtitle streams respectively. This is incorrect, as it already was before. It doesn't disable *automatic* stream

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On 27-05-2018 02:37 AM, Lou Logan wrote: On Sat, May 26, 2018, at 12:33 AM, Gyan Doshi wrote: Part of the ugliness is due to how the code fragments are rendered: with vertical margins, creating uneven line spacing. Do you mind if I reduce or eliminate those margins? Which file sets those

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On Sat, May 26, 2018, at 12:33 AM, Gyan Doshi wrote: > > Part of the ugliness is due to how the code fragments are rendered: with > vertical margins, creating uneven line spacing. Do you mind if I reduce > or eliminate those margins? Which file sets those margins? The ugly justified text in the

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

Hi Lou, Most sectioning and formatting suggestions applied locally. On 26-05-2018 03:50 AM, Lou Logan wrote: Examples are often * itemized by convention, but the resulting text justification such verbose examples makes them hard to read in HTML format. I would ditch the itemization. Part of

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On Thu, 24 May 2018 19:18:50 +0530 Gyan Doshi wrote: > From a7ff32bdac54a1f257482526a507ab7823cafb41 Mon Sep 17 00:00:00 2001 > From: Gyan Doshi > Date: Thu, 24 May 2018 19:11:00 +0530 > Subject: [PATCH] doc/ffmpeg - rewrite Stream Selection chapter Nit:

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On 24-05-2018 09:58 PM, Carl Eugen Hoyos wrote: Because of the length of the new description I wonder if the existing documentation should be kept as a "short version" or similar. Parts of the short version which are accurate have been retained; other parts are incomplete or misleading and

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On 25-05-2018 06:08 AM, Michael Niedermayer wrote: This is very detailed. I think this could byte us if we want to change some of the logic. The current chapter already does. Many askers at SE sites and even on the User ML show confusion over command line operation, including stream

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

On Thu, May 24, 2018 at 07:18:50PM +0530, Gyan Doshi wrote: > Stream Selection chapter rewritten to be more comprehensive and > illustrative. This is very detailed. I think this could byte us if we want to change some of the logic. Maybe this or some of this should be marked as being a

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

2018-05-24 15:48 GMT+02:00, Gyan Doshi : > Stream Selection chapter rewritten to be more > comprehensive and illustrative. Because of the length of the new description I wonder if the existing documentation should be kept as a "short version" or similar. Carl Eugen

[FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

Stream Selection chapter rewritten to be more comprehensive and illustrative. From a7ff32bdac54a1f257482526a507ab7823cafb41 Mon Sep 17 00:00:00 2001 From: Gyan Doshi Date: Thu, 24 May 2018 19:11:00 +0530 Subject: [PATCH] doc/ffmpeg - rewrite Stream Selection chapter Flesh out