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