Script to get committed patches

Francois Gouget fgouget at free.fr
Tue May 1 18:59:04 CDT 2001


On Wed, 2 May 2001, Simeon Pilgrim wrote:
[...]
> try to avoid the style: 
> FIXME(xxx, "(fd=%d, file=%s): stub\n", fd, name);
>                              
> but use: 
> FIXME(xxx, "(fd=%d, file=%s): stub\n", fd, name);
>                              
> The reason is that if you want to grep for things, you would search 
> for FIXME but in the first case there is no additional information
> available, where in the second one, there is (e.g. the word stub) 
> 
> But both line are the same, I'm guessing the first was ment to look 
> like:
> 
> FIXME(xxx, "(fd=%d, file=%s)\n", fd, name);

   I think you're right. I sent a patch to wine-patches.
See, fixing the documentation is simple :-)

Changelog:

   Simeon Pilgrim <simeon.pilgrim at alliedtelesyn.co.nz>

 * documentation/debugging.sgml
   Typo in stub-reporting style description


--
Francois Gouget         fgouget at free.fr        http://fgouget.free.fr/
      Broadcast message : fin du monde dans cinq minutes, repentez vous !
-------------- next part --------------
Index: debugging.sgml
===================================================================
RCS file: /home/cvs/wine/wine/documentation/debugging.sgml,v
retrieving revision 1.2
diff -u -r1.2 debugging.sgml
--- debugging.sgml	2000/12/13 21:52:37	1.2
+++ debugging.sgml	2001/05/01 23:51:00
@@ -697,7 +697,7 @@
             <para>
               try to avoid the style:
               <programlisting>
-FIXME(xxx, "(fd=%d, file=%s): stub\n", fd, name);
+FIXME(xxx, "(fd=%d, file=%s)\n", fd, name);
               </programlisting>
               but use:
               <programlisting>


More information about the wine-devel mailing list