Age | Commit message (Collapse) | Author |
|
Fix up from 5d367cd9d0c. No functional changes.
Signed-off-by: Matt Turner <mattst88@gmail.com>
|
|
Same fix appeared in libXaw as part of XFree86 4.3.0.1.
Fixes: http://gnats.netbsd.org/37610
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
Signed-off-by: Matt Turner <mattst88@gmail.com>
|
|
Patch found at http://gitorious.org/xaw3d/xaw3d/commit/ccb1ffde090006f5a715c1e3d4da5f4d6fae05d3
Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>
|
|
Found at http://gitorious.org/xaw3d/xaw3d/commit/3ba3e52454bb684a73601ec91e2c274e776f711a
Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>
|
|
This ugly piece of magic was copied from libXaw, and now scrollbars
will work slightly less incorrectly than they did before.
Found at http://gitorious.org/xaw3d/xaw3d/commit/4ec68aebffbdc406139f880f92b6c18b835af6d4
Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>
|
|
Fixes segmentation fault when closing gv.
Found at http://gitorious.org/xaw3d/xaw3d/commit/23b3ca50395f180cba4a923eb8827dcc961629da
Fixes: https://bugs.gentoo.org/show_bug.cgi?id=406487
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
Signed-off-by: Matt Turner <mattst88@gmail.com>
|
|
Introduced in f6cd786d1.
Reported-by: Jean-Pierre Demailly <Jean-Pierre.Demailly@ujf-grenoble.fr>
Signed-off-by: Matt Turner <mattst88@gmail.com>
|
|
AsciiText.h doesn't include MultiSrc.h when internationalization is
disabled, so dcd27a9c69a was breaking the build. Also fix a typo
introduced in the same commit Xaw -> Xaw3d.
Signed-off-by: Matt Turner <mattst88@gmail.com>
|
|
When XAW_INTERNATIONALIZATION isn't defined, some uninitialized stack
data was being passed to XtGetValues. This makes it not do that.
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=46124
Reviewed-by: Matt Turner <mattst88@gmail.com>
Signed-off-by: Devin J. Pohly <djpohly+xorg@gmail.com>
|
|
Noticed when comparing with libXaw's AllWidgets.c.
|
|
Makes these two files closer to the libXaw originals.
Signed-off-by: Matt Turner <mattst88@gmail.com>
|
|
Caught a mistake in SmeThreeD.h: smeThreeDWidgetClass was declared
instead of smeThreeDObjectClass.
Also fixes redundant redeclaration warnings.
Signed-off-by: Matt Turner <mattst88@gmail.com>
|
|
Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>
|
|
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>
|
|
Tip.c:336:9: error: implicit declaration of function 'abs' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
ksy += abs(ext->max_ink_extent.y);
^
1 error generated.
Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>
|
|
Was toupper() *really* expensive when this was written? :)
Also, initialize action_type with toupper(), since the check immediately
following assumes that its value is uppercase.
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Matt Turner <mattst88@gmail.com>
|
|
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Matt Turner <mattst88@gmail.com>
|
|
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Matt Turner <mattst88@gmail.com>
|
|
Fixes gcc warning:
Scrollbar.c: In function 'NotifyScroll':
Scrollbar.c:950:12: warning: 'call_data' may be used uninitialized in this function [-Wuninitialized]
Reviewed-by: Matt Turner <mattst88@gmail.com>
Signed-off-by: Guillem Jover <guillem@hadrons.org>
|
|
Fixes gcc warnings:
Scrollbar.c: In function 'NotifyScroll':
Scrollbar.c:955:37: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
Text.c: In function 'HScroll':
Text.c:1274:41: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
Text.c: In function 'VScroll':
Text.c:1431:31: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
TextPop.c: In function 'DoSearch':
TextPop.c:808:31: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
TextPop.c: In function 'Replace':
TextPop.c:942:31: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
Viewport.c: In function 'ScrollUpDownProc':
Viewport.c:866:15: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
Reviewed-by: Matt Turner <mattst88@gmail.com>
Signed-off-by: Guillem Jover <guillem@hadrons.org>
|
|
Fixes gcc warning:
Scrollbar.c: In function 'MoveThumb':
Scrollbar.c:1006:16: warning: variable 't' set but not used [-Wunused-but-set-variable]
Reviewed-by: Matt Turner <mattst88@gmail.com>
Signed-off-by: Guillem Jover <guillem@hadrons.org>
|
|
Fixes gcc warning:
laygram.c: In function 'LayYYparse':
laygram.y:242:7: warning: label 'unary' defined but not used [-Wunused-label]
Reviewed-by: Matt Turner <mattst88@gmail.com>
Signed-off-by: Guillem Jover <guillem@hadrons.org>
|
|
Fixes gcc warnings:
laylex.c:1316:17: warning: 'yyunput' defined but not used [-Wunused-function]
laylex.c:1357:16: warning: 'input' defined but not used [-Wunused-function]
Reviewed-by: Matt Turner <mattst88@gmail.com>
Signed-off-by: Guillem Jover <guillem@hadrons.org>
|
|
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Matt Turner <mattst88@gmail.com>
|
|
Functions were only called inside ifdef blocks, but were defined
outside.
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Matt Turner <mattst88@gmail.com>
|
|
Fixes gcc warning:
laygram.y:259:5: warning: no previous prototype for 'LayYYwrap'
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
|
|
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@oracle.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
|
|
The -d flag is used for bison/yacc and has the same effect.
Signed-off-by: Matthieu Herrb <matthieu.herrb@laas.fr>
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Fixes compiler warning:
"AsciiSrc.c", line 858: warning: argument #2 is incompatible with prototype:
prototype: pointer to char : "AsciiSrc.c", line 118
argument : pointer to const char
Has to use _XtString (#define char *) instead of String (typedef char *) to
correctly be a pointer to const chars instead of a const pointer to chars.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
|
|
Ensures all sources use consistent types and provides central place
to add _X_HIDDEN tags to avoid exporting these as part of the API/ABI.
Also fixes:
"laygram.c", line 1363: warning: implicit function declaration: LayYYlex
"laygram.c", line 1777: warning: implicit function declaration: LayYYerror
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
|
|
Oops. Helps to include the docs in the tarball if you want to install them.
Reported-by: Gaetan Nadon <memsize@videotron.ca>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Gaetan Nadon <memsize@videotron.ca>
|
|
List X.Org as current maintainers instead of D. J. Hawkey Jr.
Replace Imakefile configuration instructions with configure flags
Remove sections about building in an old X11R5 or R6 monolithic tree.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
|
|
The previous maintainer strongly recommended providing this file with
the library as the authoritative documentation for the libXaw3d API.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
|
|
The %name-prefix "prefix" syntax was introduced in bison 2.3b and
is not supported in older bison and yacc parsers.
Without this fix, bison 2.3 (older than 2.3b) gives the cryptic error:
.../src/laygram.y:1.14-20: syntax error, unexpected string, expecting =
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Tested-by: Gaetan Nadon <memsize@videotron.ca>
|
|
rather than #include "TemplateP.h".
Acked-by: Matt Turner <mattst88@gmail.com>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
|
|
Expections: TexTr.c, sharedlib.c ans generated lex and yacc C code.
XawI18n.h needs HAVE_WCTYPE_H and HAVE_WCHAR_H.
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
|
|
Reviewed-by: Gaetan Nadon <memsize@videotron.ca>
Tested-by: Gaetan Nadon <memsize@videotron.ca>
Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
|
|
Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
Acked-by: Matt Turner <mattst88@gmail.com>
|
|
AsciiSink.c:132:41: warning: initialization from incompatible pointer type
AsciiSink.c:135:41: warning: initialization from incompatible pointer type
AsciiSink.c:137:41: warning: initialization from incompatible pointer type
Vendor.c:199:24: warning: initialization from incompatible pointer type
Vendor.c:269:24: warning: initialization from incompatible pointer type
Vendor.c:284:24: warning: initialization from incompatible pointer type
Viewport.c:143:3: warning: initialization from incompatible pointer type
MultiSink.c:160:41: warning: initialization from incompatible pointer type
MultiSink.c:162:41: warning: initialization from incompatible pointer type
Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
Acked-by: Matt Turner <mattst88@gmail.com>
|
|
Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
Acked-by: Gaetan Nadon <memsize@videotron.ca>
Acked-by: Matt Turner <mattst88@gmail.com>
|
|
Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
Acked-by: Gaetan Nadon <memsize@videotron.ca>
Acked-by: Matt Turner <mattst88@gmail.com>
|
|
Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
Acked-by: Gaetan Nadon <memsize@videotron.ca>
Acked-by: Matt Turner <mattst88@gmail.com>
|
|
All published Xaw3d header file are included this way.
The -I directive to include/X11/Xaw3d is removed which will alert
developers not to include header files with quotes unless they are in
the /src directory.
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
|
|
This will also prevent confusion as it is not obtained from PKG_CHECK_MODULES
but follows its naming convention.
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
|
|
AM_CPPFLAGS should only contain preprocessor directives like -I -D.
The various *_CFLAGS may contain compiler flags as they are obtained
from pkg-config Cflags variable.
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
|
|
Automake will handle the distribution of laygram.h
http://www.gnu.org/software/automake/manual/automake.html#Yacc-and-Lex
http://cgit.freedesktop.org/xorg/app/twm/tree/src/Makefile.am
http://cgit.freedesktop.org/xorg/app/xgc/tree/Makefile.am
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
|
|
Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
Acked-by: Gaetan Nadon <memsize@videotron.ca>
|
|
Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
Acked-by: Gaetan Nadon <memsize@videotron.ca>
|
|
Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
Acked-by: Gaetan Nadon <memsize@videotron.ca>
|
|
Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
Acked-by: Gaetan Nadon <memsize@videotron.ca>
|