From joel@automatvapen.se  Fri Sep 17 18:27:23 2004
Return-Path: <joel@automatvapen.se>
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 9D2E416A4D0
	for <FreeBSD-gnats-submit@freebsd.org>; Fri, 17 Sep 2004 18:27:23 +0000 (GMT)
Received: from av8-2-sn3.vrr.skanova.net (av8-2-sn3.vrr.skanova.net [81.228.9.184])
	by mx1.FreeBSD.org (Postfix) with ESMTP id D2C1343D4C
	for <FreeBSD-gnats-submit@freebsd.org>; Fri, 17 Sep 2004 18:27:22 +0000 (GMT)
	(envelope-from joel@automatvapen.se)
Received: by av8-2-sn3.vrr.skanova.net (Postfix, from userid 502)
	id 8CE463836F; Fri, 17 Sep 2004 20:27:21 +0200 (CEST)
Received: from smtp3-2-sn3.vrr.skanova.net (smtp3-2-sn3.vrr.skanova.net [81.228.9.102])
	by av8-2-sn3.vrr.skanova.net (Postfix) with ESMTP id 7AEDD37EAB
	for <FreeBSD-gnats-submit@freebsd.org>; Fri, 17 Sep 2004 20:27:21 +0200 (CEST)
Received: from dude.automatvapen.se (t2o955p72.telia.com [195.252.52.192])
	by smtp3-2-sn3.vrr.skanova.net (Postfix) with SMTP id 70A7F37E42
	for <FreeBSD-gnats-submit@freebsd.org>; Fri, 17 Sep 2004 20:27:20 +0200 (CEST)
Received: by dude.automatvapen.se (sSMTP sendmail emulation); Fri, 17 Sep 2004 20:27:45 +0200
Message-Id: <20040917182720.70A7F37E42@smtp3-2-sn3.vrr.skanova.net>
Date: Fri, 17 Sep 2004 20:27:45 +0200
From: "Joel Dahl" <joel@automatvapen.se>
Reply-To: Joel Dahl <joel@automatvapen.se>
To: FreeBSD-gnats-submit@freebsd.org
Cc:
Subject: [PATCH] Whitespace cleanup
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         71826
>Category:       docs
>Synopsis:       [PATCH] Whitespace cleanup
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-doc
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          doc-bug
>Submitter-Id:   current-users
>Arrival-Date:   Fri Sep 17 18:30:10 GMT 2004
>Closed-Date:    Fri Oct 01 18:47:14 GMT 2004
>Last-Modified:  Fri Oct 01 18:47:14 GMT 2004
>Originator:     Joel Dahl
>Release:        FreeBSD 5.2.1-RELEASE-p9 i386
>Organization:
>Environment:
System: FreeBSD dude.automatvapen.se 5.2.1-RELEASE-p9 FreeBSD 5.2.1-RELEASE-p9 #1: Sat Jul 10 16:52:28 CEST 2004 root@dude.automatvapen.se:/usr/obj/usr/src/sys/WORKSTATION i386


	
>Description:
The attached patch fixes some whitespaces in
doc/en_US.ISO8859-1/books/handbook/multimedia/chapter.sgml
	

>How-To-Repeat:
	
>Fix:

	

--- whitespace.diff begins here ---
--- chapter.sgml	Fri Sep 17 19:45:34 2004
+++ new.chapter.sgml	Fri Sep 17 20:18:48 2004
@@ -127,7 +127,7 @@
     </sect1info>
 
     <title>Setting Up the Sound Card</title>
-    
+
   <sect2 id="sound-device">
     <title>Configuring the System</title>
 
@@ -836,7 +836,7 @@
   Adaptor #0: "Savage Streams Engine"
     number of ports: 1
     port base: 43
-    operations supported: PutImage 
+    operations supported: PutImage
     supported visuals:
       depth 16, visualID 0x22
       depth 16, visualID 0x23
@@ -1247,7 +1247,7 @@
       same time, takes some of the more fine-grained control away from
       the user.  The <application>xine</application> video player
       performs best on XVideo interfaces.</para>
-     
+
     <para>By default, <application>xine</application> player will
     start up in a graphical user interface.  The menus can then be
     used to open a specific file:</para>
@@ -1327,7 +1327,7 @@
      and use a few different applications.  This section exists to
       give the reader pointers to such additional information.</para>
 
-    <para>The 
+    <para>The
       <ulink url="http://www.mplayerhq.hu/DOCS/">MPlayer documentation</ulink>
       is very technically informative.
       These documents should probably be consulted by anyone wishing
@@ -1427,7 +1427,7 @@
       <para>Alternatively, you may statically compile the support for
 	the TV card in your kernel, in that case add the following
 	lines to your kernel configuration:</para>
-	
+
       <programlisting>device	 bktr
 device	iicbus
 device	iicbb
--- whitespace.diff ends here ---


>Release-Note:
>Audit-Trail:

From: Marc Fonvieille <blackend@FreeBSD.org>
To: Joel Dahl <joel@automatvapen.se>
Cc: FreeBSD-gnats-submit@FreeBSD.org
Subject: Re: docs/71826: [PATCH] Whitespace cleanup
Date: Fri, 17 Sep 2004 20:42:45 +0200

 On Fri, Sep 17, 2004 at 08:27:45PM +0200, Joel Dahl wrote:
 > >Description:
 > The attached patch fixes some whitespaces in
 > doc/en_US.ISO8859-1/books/handbook/multimedia/chapter.sgml
 >
 
 This was often discussed in past (consult the freebsd-doc archives).
 With often the same result: when it's not "mandatory", we leave the
 things in this state.
 
 Marc

From: Joel Dahl <joel@automatvapen.se>
To: Marc Fonvieille <blackend@FreeBSD.org>
Cc: FreeBSD-gnats-submit@FreeBSD.org
Subject: Re: docs/71826: [PATCH] Whitespace cleanup
Date: Fri, 17 Sep 2004 21:19:46 +0200

 On Fri, 2004-09-17 at 20:42, Marc Fonvieille wrote:
 > On Fri, Sep 17, 2004 at 08:27:45PM +0200, Joel Dahl wrote:
 > > >Description:
 > > The attached patch fixes some whitespaces in
 > > doc/en_US.ISO8859-1/books/handbook/multimedia/chapter.sgml
 > >
 > 
 > This was often discussed in past (consult the freebsd-doc archives).
 > With often the same result: when it's not "mandatory", we leave the
 > things in this state.
 > 
 > Marc
 
 Hm, yes, I just found a couple of threads where this was discussed.
 
 Joel
 

From: Joel Dahl <joel@automatvapen.se>
To: Marc Fonvieille <blackend@FreeBSD.org>
Cc: FreeBSD-gnats-submit@FreeBSD.org
Subject: Re: docs/71826: [PATCH] Whitespace cleanup
Date: Sat, 18 Sep 2004 09:05:42 +0200

 While here, just one small note about "7.4.2.1.1 Building MPlayer":
 
 "However, if you need the XviD codec, you have to specify the WITH_XVID
 option in the command line."
 
 Should be:
 
 "However, if you need the XviD codec, you have to specify the WITH_XVID
 option on the command line."
 
 
 Joel
 

From: Marc Fonvieille <blackend@FreeBSD.org>
To: Joel Dahl <joel@automatvapen.se>
Cc: FreeBSD-gnats-submit@FreeBSD.org
Subject: Re: docs/71826: [PATCH] Whitespace cleanup
Date: Sat, 18 Sep 2004 09:34:21 +0200

 On Sat, Sep 18, 2004 at 09:05:42AM +0200, Joel Dahl wrote:
 > While here, just one small note about "7.4.2.1.1 Building MPlayer":
 > 
 > "However, if you need the XviD codec, you have to specify the WITH_XVID
 > option in the command line."
 > 
 > Should be:
 > 
 > "However, if you need the XviD codec, you have to specify the WITH_XVID
 > option on the command line."
 > 
 >
 
 Well Please could you use the right place for that?
 
 You added in the audit trail something which has nothing to do with the PR.
 And btw I'm not guilty of the whitespaces :))
 
 Please try to use the PR followup in the right way, Thanks.
 
 Marc

From: Ceri Davies <ceri@submonkey.net>
To: Marc Fonvieille <blackend@FreeBSD.org>
Cc: freebsd-doc@FreeBSD.org
Subject: Re: docs/71826: [PATCH] Whitespace cleanup
Date: Mon, 20 Sep 2004 18:15:14 +0100

 --MZf7D3rAEoQgPanC
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: inline
 Content-Transfer-Encoding: quoted-printable
 
 On Fri, Sep 17, 2004 at 06:50:11PM +0000, Marc Fonvieille wrote:
 > The following reply was made to PR docs/71826; it has been noted by GNATS.
 >=20
 > From: Marc Fonvieille <blackend@FreeBSD.org>
 > To: Joel Dahl <joel@automatvapen.se>
 > Cc: FreeBSD-gnats-submit@FreeBSD.org
 > Subject: Re: docs/71826: [PATCH] Whitespace cleanup
 > Date: Fri, 17 Sep 2004 20:42:45 +0200
 >=20
 >  On Fri, Sep 17, 2004 at 08:27:45PM +0200, Joel Dahl wrote:
 >  > >Description:
 >  > The attached patch fixes some whitespaces in
 >  > doc/en_US.ISO8859-1/books/handbook/multimedia/chapter.sgml
 >  >
 > =20
 >  This was often discussed in past (consult the freebsd-doc archives).
 >  With often the same result: when it's not "mandatory", we leave the
 >  things in this state.
 
 If this is the consensual view (I don't recall the discussion), then
 please close the PR.
 
 Ceri
 --=20
 It is not tinfoil, it is my new skin.  I am a robot.
 
 --MZf7D3rAEoQgPanC
 Content-Type: application/pgp-signature
 Content-Disposition: inline
 
 -----BEGIN PGP SIGNATURE-----
 Version: GnuPG v1.2.6 (FreeBSD)
 
 iD8DBQFBTxAiocfcwTS3JF8RAnxSAKCMy6O2q6uy6sGUvWZfgOwUbJ+PPQCeLsIf
 YtkCtnTCiW49Ekcn/0Kj/Ig=
 =h3Yc
 -----END PGP SIGNATURE-----
 
 --MZf7D3rAEoQgPanC--

From: Giorgos Keramidas <keramida@freebsd.org>
To: Ceri Davies <ceri@freebsd.org>,
	Marc Fonvieille <blackend@freebsd.org>
Cc: bug-followup@freebsd.org
Subject: Re: docs/71826: [PATCH] Whitespace cleanup
Date: Tue, 21 Sep 2004 08:54:45 +0300

 On 2004-09-20 18:15, Ceri Davies <ceri@submonkey.net> wrote:
 > On Fri, Sep 17, 2004 at 06:50:11PM +0000, Marc Fonvieille wrote:
 > >  > The attached patch fixes some whitespaces in
 > >  > doc/en_US.ISO8859-1/books/handbook/multimedia/chapter.sgml
 > >
 > >  This was often discussed in past (consult the freebsd-doc archives).
 > >  With often the same result: when it's not "mandatory", we leave the
 > >  things in this state.
 >
 > If this is the consensual view (I don't recall the discussion), then
 > please close the PR.
 
 Every commit increases the disk space the particular file needs on the CVS
 repository server, on all the CVS mirrors, on all the copies of the tree that
 users keep privately at their home or work, etc.  Bearing this in mind, I have
 to agree with Marc.  Whitespace commits should only be done rarely, if there's
 no real problem they might solve (such as aesthetic issues of a particular
 output format, wrapping of output documents, or similar).
 
 Yes, please close this PR.  When someone does a few content commits to the
 multimedia chapter a whitespace commit can follow.  Fixing spaces just for the
 sake of fixing spaces isn't really worth the disk space it will take.
 
 - Giorgos
 
State-Changed-From-To: open->closed 
State-Changed-By: ceri 
State-Changed-When: Fri Oct 1 18:46:47 GMT 2004 
State-Changed-Why:  
It was decided that this patch is unwanted. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=71826 
>Unformatted:
