diff mbox

DocBook: grammatical correction on DVB Overview

Message ID 20150206181752.GA25234@biggie (mailing list archive)
State Changes Requested, archived
Headers

Commit Message

Luis de Bethencourt Feb. 6, 2015, 6:17 p.m. UTC 
  ---
 Documentation/DocBook/media/dvb/intro.xml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Jonathan Corbet Feb. 6, 2015, 6:23 p.m. UTC | #1 
On Fri, 6 Feb 2015 18:17:52 +0000
Luis de Bethencourt <luis@debethencourt.com> wrote:

> -video streams. Besides usually several of such audio and video streams
> -it also contains data streams with information about the programs
> +video streams. As well as several of such audio and video streams, it
> +usually also contains data streams with information about the programs

Not sure if I see this as an improvement or not; you've changed the
meaning of the sentence a bit.  It also lacks changelog and signoff...

Thanks,

jon
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Luis de Bethencourt Feb. 6, 2015, 6:27 p.m. UTC | #2 
On Fri, Feb 06, 2015 at 01:23:15PM -0500, Jonathan Corbet wrote:
> On Fri, 6 Feb 2015 18:17:52 +0000
> Luis de Bethencourt <luis@debethencourt.com> wrote:
> 
> > -video streams. Besides usually several of such audio and video streams
> > -it also contains data streams with information about the programs
> > +video streams. As well as several of such audio and video streams, it
> > +usually also contains data streams with information about the programs
> 
> Not sure if I see this as an improvement or not; you've changed the
> meaning of the sentence a bit.  It also lacks changelog and signoff...
> 
> Thanks,
> 
> jon

Hi Jon,

The original sentence is hard to read, I had to go a few times before I think
I understood what it meant. Which might be different from your interpretation.

How about?:
"Besides several of such audio and video stream, it usually also contains data
streams with information about the programs [...]"

If this isn't worth it and it is just nitpicking, feel free to drop.

Thanks for taking the time to look into this,
Luis
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/Documentation/DocBook/media/dvb/intro.xml b/Documentation/DocBook/media/dvb/intro.xml
index 2048b53..e98bfa9 100644
--- a/Documentation/DocBook/media/dvb/intro.xml
+++ b/Documentation/DocBook/media/dvb/intro.xml
@@ -85,8 +85,8 @@  real time and re-inserted into the TS.</para>
  <para>Demultiplexer which filters the incoming DVB stream</para>
 
 <para>The demultiplexer splits the TS into its components like audio and
-video streams. Besides usually several of such audio and video streams
-it also contains data streams with information about the programs
+video streams. As well as several of such audio and video streams, it
+usually also contains data streams with information about the programs
 offered in this or other streams of the same provider.</para>
 
 </listitem>