[PATCH:libXaw3d 1/2] Fix several unused variable warnings

Alan Coopersmith alan.coopersmith at oracle.com
Sat Sep 17 00:57:25 PDT 2011


All but one are due to variables defined outside #ifdef's but only
used inside #ifdef'ed code blocks.

Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
---
 src/Label.c      |    2 ++
 src/SmeBSB.c     |    2 +-
 src/Text.c       |    2 +-
 src/TextAction.c |   10 ++++++----
 4 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/src/Label.c b/src/Label.c
index c3f8e17..46b0465 100644
--- a/src/Label.c
+++ b/src/Label.c
@@ -528,7 +528,9 @@ Redisplay(Widget gw, XEvent *event, Region region)
 	int len = w->label.label_len;
 	char *label = w->label.label;
 	Position y = w->label.label_y + w->label.font->max_bounds.ascent;
+#ifdef XAW_INTERNATIONALIZATION
         Position ksy = w->label.label_y;
+#endif
 
 	/* display left bitmap */
 	if (w->label.left_bitmap && w->label.lbm_width != 0) {
diff --git a/src/SmeBSB.c b/src/SmeBSB.c
index 5a18003..f8bf987 100644
--- a/src/SmeBSB.c
+++ b/src/SmeBSB.c
@@ -475,7 +475,7 @@ static XtGeometryResult
 QueryGeometry(Widget w, XtWidgetGeometry *intended, XtWidgetGeometry *return_val)
 {
     SmeBSBObject entry = (SmeBSBObject) w;
-    Dimension width, height, h;
+    Dimension width, height;
     XtGeometryResult ret_val = XtGeometryYes;
     XtGeometryMask mode = intended->request_mode;
 
diff --git a/src/Text.c b/src/Text.c
index 1873016..36bef53 100644
--- a/src/Text.c
+++ b/src/Text.c
@@ -804,12 +804,12 @@ _XawTextGetSTRING(TextWidget ctx, XawTextPosition left, XawTextPosition right)
   unsigned char *s;
   unsigned char c;
   long i, j, n;
-  wchar_t *ws, wc;
 
   /* allow ESC in accordance with ICCCM */
 #ifdef XAW_INTERNATIONALIZATION
   if (_XawTextFormat(ctx) == XawFmtWide) {
      MultiSinkObject sink = (MultiSinkObject) ctx->text.sink;
+     wchar_t *ws, wc;
      ws = (wchar_t *)_XawTextGetText(ctx, left, right);
      n = wcslen(ws);
      for (j = 0, i = 0; j < n; j++) {
diff --git a/src/TextAction.c b/src/TextAction.c
index 1de645a..7d1afcb 100644
--- a/src/TextAction.c
+++ b/src/TextAction.c
@@ -1278,9 +1278,9 @@ TextFocusOut(Widget w, XEvent *event, String *p, Cardinal *n)
 static void
 TextEnterWindow(Widget w, XEvent *event, String *params, Cardinal *num_params)
 {
+#ifdef XAW_INTERNATIONALIZATION
   TextWidget ctx = (TextWidget) w;
 
-#ifdef XAW_INTERNATIONALIZATION
   if ((event->xcrossing.detail != NotifyInferior) && event->xcrossing.focus &&
       !ctx->text.hasfocus) {
 	_XawImSetFocusValues(w, NULL, 0);
@@ -1292,9 +1292,9 @@ TextEnterWindow(Widget w, XEvent *event, String *params, Cardinal *num_params)
 static void
 TextLeaveWindow(Widget w, XEvent *event, String *params, Cardinal *num_params)
 {
+#ifdef XAW_INTERNATIONALIZATION
   TextWidget ctx = (TextWidget) w;
 
-#ifdef XAW_INTERNATIONALIZATION
   if ((event->xcrossing.detail != NotifyInferior) && event->xcrossing.focus &&
       !ctx->text.hasfocus) {
 	_XawImUnsetFocus(w);
@@ -1509,7 +1509,9 @@ static void
 InsertString(Widget w, XEvent *event, String *params, Cardinal *num_params)
 {
   TextWidget ctx = (TextWidget) w;
+#ifdef XAW_INTERNATIONALIZATION
   XtAppContext app_con = XtWidgetToApplicationContext(w);
+#endif
   XawTextBlock text;
   int	   i;
 
@@ -1663,7 +1665,6 @@ StripOutOldCRs(TextWidget ctx, XawTextPosition from, XawTextPosition to)
   Widget src = ctx->text.source;
   XawTextBlock text;
   char *buf;
-  static wchar_t wc_two_spaces[ 3 ];
 
   /* Initialize our TextBlock with two spaces. */
 
@@ -1673,6 +1674,7 @@ StripOutOldCRs(TextWidget ctx, XawTextPosition from, XawTextPosition to)
       text.ptr= "  ";
 #ifdef XAW_INTERNATIONALIZATION
   else {
+      static wchar_t wc_two_spaces[ 3 ];
       wc_two_spaces[0] = _Xaw_atowc(XawSP);
       wc_two_spaces[1] = _Xaw_atowc(XawSP);
       wc_two_spaces[2] = 0;
@@ -1761,7 +1763,6 @@ InsertNewCRs(TextWidget ctx, XawTextPosition from, XawTextPosition to)
   XawTextBlock text;
   int i, width, height, len;
   char * buf;
-  static wchar_t wide_CR[ 2 ];
 
   text.firstPos = 0;
   text.length = 1;
@@ -1771,6 +1772,7 @@ InsertNewCRs(TextWidget ctx, XawTextPosition from, XawTextPosition to)
       text.ptr = "\n";
 #ifdef XAW_INTERNATIONALIZATION
   else {
+      static wchar_t wide_CR[ 2 ];
       wide_CR[0] = _Xaw_atowc(XawLF);
       wide_CR[1] = 0;
       text.ptr = (char*) wide_CR;
-- 
1.7.3.2



More information about the xorg-devel mailing list