diff options
author | Jeremy Huddleston <jeremyhu@apple.com> | 2012-01-22 11:58:09 -0800 |
---|---|---|
committer | Jeremy Huddleston <jeremyhu@apple.com> | 2012-01-22 12:01:18 -0800 |
commit | 702e4ff9f5fbfae4aecba09f8c735dc97bac1b59 (patch) | |
tree | 01f404606c1b02f79d3d156d039eac40abf84ec9 | |
parent | d31153b30abd26c3f65e19d126ce4a3542cf14c0 (diff) |
Cleanup shadow declarations
Box.c:265:10: warning: declaration shadows a local variable [-Wshadow]
int i = bbw->composite.num_children;
^
Box.c:166:15: note: previous declaration is here
Cardinal i;
^
1 warning generated.
Form.c:809:9: warning: declaration shadows a local variable [-Wshadow]
Widget w = *childP;
^
Form.c:793:24: note: previous declaration is here
XawFormDoLayout(Widget w,
^
1 warning generated.
Viewport.c:627:24: warning: declaration shadows a local variable [-Wshadow]
if (!needshoriz) CheckHoriz();
^
Viewport.c:605:14: note: expanded from macro 'CheckHoriz'
Widget bar = w->viewport.horiz_bar; \
^
Viewport.c:620:14: note: previous declaration is here
Widget bar = w->viewport.vert_bar;
^
1 warning generated.
Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>
-rw-r--r-- | src/Box.c | 4 | ||||
-rw-r--r-- | src/Form.c | 6 | ||||
-rw-r--r-- | src/Viewport.c | 20 |
3 files changed, 15 insertions, 15 deletions
@@ -262,9 +262,9 @@ DoLayout(BoxWidget bbw, Dimension width, Dimension height, if (bbw->composite.num_children == num_mapped_children) XMapSubwindows( XtDisplay((Widget)bbw), XtWindow((Widget)bbw) ); else { - int i = bbw->composite.num_children; + int j = bbw->composite.num_children; Widget *childP = bbw->composite.children; - for (; i > 0; childP++, i--) + for (; j > 0; childP++, j--) if (XtIsRealized(*childP) && XtIsManaged(*childP) && (*childP)->core.mapped_when_managed) XtMapWidget(*childP); @@ -790,7 +790,7 @@ PreferredGeometry(Widget widget, XtWidgetGeometry *request, XtWidgetGeometry *re */ void -XawFormDoLayout(Widget w, +XawFormDoLayout(Widget _fw, #if NeedWidePrototypes int doit) #else @@ -798,11 +798,11 @@ XawFormDoLayout(Widget w, #endif { Widget *childP; - FormWidget fw = (FormWidget)w; + FormWidget fw = (FormWidget)_fw; int num_children = fw->composite.num_children; WidgetList children = fw->composite.children; - if ( ((fw->form.no_refigure = !doit) == TRUE) || !XtIsRealized(w) ) + if ( ((fw->form.no_refigure = !doit) == TRUE) || !XtIsRealized(_fw) ) return; for (childP = children; childP - children < num_children; childP++) { diff --git a/src/Viewport.c b/src/Viewport.c index abb8961..a37a9a1 100644 --- a/src/Viewport.c +++ b/src/Viewport.c @@ -602,12 +602,12 @@ ComputeLayout(Widget widget, Boolean query, Boolean destroy_scrollbars) if (w->viewport.allowhoriz && \ (int)preferred.width > clip_width + 2 * sw) { \ if (!needshoriz) { \ - Widget bar = w->viewport.horiz_bar; \ + Widget horiz_bar = w->viewport.horiz_bar; \ needshoriz = True; \ - if (bar == (Widget)NULL) \ - bar = CreateScrollbar(w, True); \ - clip_height -= bar->core.height + \ - bar->core.border_width + pad; \ + if (horiz_bar == (Widget)NULL) \ + horiz_bar = CreateScrollbar(w, True); \ + clip_height -= horiz_bar->core.height + \ + horiz_bar->core.border_width + pad; \ if (clip_height < 1) clip_height = 1; \ } \ intended.width = preferred.width; \ @@ -617,12 +617,12 @@ ComputeLayout(Widget widget, Boolean query, Boolean destroy_scrollbars) if (w->viewport.allowvert && (int)preferred.height > clip_height + 2 * sw) { if (!needsvert) { - Widget bar = w->viewport.vert_bar; + Widget vert_bar = w->viewport.vert_bar; needsvert = True; - if (bar == (Widget)NULL) - bar = CreateScrollbar(w, False); - clip_width -= bar->core.width + - bar->core.border_width + pad; + if (vert_bar == (Widget)NULL) + vert_bar = CreateScrollbar(w, False); + clip_width -= vert_bar->core.width + + vert_bar->core.border_width + pad; if (clip_width < 1) clip_width = 1; if (!needshoriz) CheckHoriz(); } |