On Wed, Feb 24, 2010 at 6:08 PM, Colin Watson <cjwat...@debian.org> wrote:

> The uses of TRUE and FALSE only occur in a Debian patch.  Perhaps it
> would be more appropriate to simply use 1 and 0 there directly?

Right. Or should we merge the TRUE/FALSE definitions
into 020_bidi.patch directly, to keep the patch self-explained?

Regards,
-- 
Theppitak Karoonboonyanan
http://linux.thai.net/~thep/
diff -u newt-0.52.10/debian/patches/020_bidi.patch newt-0.52.10/debian/patches/020_bidi.patch
--- newt-0.52.10/debian/patches/020_bidi.patch
+++ newt-0.52.10/debian/patches/020_bidi.patch
@@ -120,7 +120,7 @@
 diff -ruN newt-0.52.10-old/newt.c newt-0.52.10/newt.c
 --- newt-0.52.10-old/newt.c	2008-07-30 13:42:55.000000000 +0100
 +++ newt-0.52.10/newt.c	2009-04-20 21:22:45.000000000 +0100
-@@ -18,6 +18,461 @@
+@@ -18,6 +18,468 @@
  #include "newt.h"
  #include "newt_pr.h"
  
@@ -128,6 +128,13 @@
 +#include <fribidi/fribidi.h>
 +#include <dlfcn.h>
 +
++#ifndef TRUE
++# define TRUE	1
++#endif
++#ifndef FALSE
++# define FALSE	0
++#endif
++
 +/* No sense in enabling shaping if we don't have BIDI support. */
 +typedef struct
 +{

Reply via email to