[FFmpeg-devel] [PATCH] doc/indevs: rework option description in the x11 grab section

Stefano Sabatini stefasab at gmail.com
Thu Aug 9 00:30:11 CEST 2012


On date Wednesday 2012-08-08 23:00:20 +0200, Clément Bœsch encoded:
> On Wed, Aug 08, 2012 at 11:21:56AM +0200, Stefano Sabatini wrote:
> > Create a dedicated options subchapter, rather than a subchapter for each
> > option.
> > 
> > This is more consistent with the rest of the docs. Also apply misc minor
> > fixes for improving rendering.
> > ---
> >  doc/indevs.texi |   35 ++++++++++++++++++----------------
> >  1 files changed, 18 insertions(+), 17 deletions(-)
[...] 
> LGTM.

Applied.
 
> Note: could be nice if you could send these patches as a patchset,
> especially since the order is important :)

The problem with this approach is that the order of patches often
changes in my local repo, dependencies between patches as well, and I
tend to add/remove/merge patches as review is done and I
rework/rethink the patchset, so it still makes sense to me to post
each patch separately (yes this also has drawbacks, but everyone is
different and adopts the workflow which suits her better).
-- 
FFmpeg = Fast & Friendly Merciful Plastic Eccentric God


More information about the ffmpeg-devel mailing list