Age | Commit message (Collapse) | Author | Files | Lines |
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
To use this in your local repo clone, you will need to either run
`git blame --ignore-revs-file .git-blame-ignore-revs`
or set it permanently with
`git config blame.ignoreRevsFile .git-blame-ignore-revs`
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
lex.c: In function ‘GetLine’:
lex.c:38:8: warning: ‘c’ may be used uninitialized [-Wmaybe-uninitialized]
38 | if (c == '\n')
| ^
lex.c:29:16: note: ‘c’ was declared here
29 | int i = 0, c;
| ^
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
USE_XFT ifdefs have been in since the import from XFree86, but
never enabled by the autoconf build, oops!
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Simple test case:
% python -c 'print("A"*60000)' > buf
% xditview buf
Reported-by: constantine110@protonmail.com
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Mostly via util/modular/x-indent-all.sh, plus some manual cleanup
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
As suggested by cppcheck:
parse.c:163:19: style: Variable 'guesses' can be declared with const [constVariable]
static int guesses[] = { 1, 4, 100, 1000, 1 };
^
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
As warned by cppcheck:
xditview.c:338:13: error: Resource leak: new_file [resourceLeak]
return;
^
xditview.c:342:13: error: Resource leak: new_file [resourceLeak]
return;
^
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
As suggested by cppcheck:
xditview.c:359:32: style: Obsolescent function 'rindex' called.
It is recommended to use 'strrchr' instead. [rindexCalled]
if (name[0] != '/' && (n = rindex (name, '/')))
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
xditview.c: In function ‘main’:
xditview.c:196:23: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (int i = 0; i < XtNumber (popupMenuEntries); i++) {
^
xditview.c:211:23: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (int i = 0; i < XtNumber (fileMenuEntries); i++) {
^
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
DviChar.c:392:1: warning: missing initializer for field ‘buckets’ of ‘DviCharNameMap {aka struct _dviCharNameMap}’ [-Wmissing-field-initializers]
};
^
In file included from DviChar.c:9:0:
DviChar.h:34:22: note: ‘buckets’ declared here
DviCharNameHash *buckets[DVI_HASH_SIZE];
^~~~~~~
DviChar.c:658:1: warning: missing initializer for field ‘buckets’ of ‘DviCharNameMap {aka struct _dviCharNameMap}’ [-Wmissing-field-initializers]
};
^
In file included from DviChar.c:9:0:
DviChar.h:34:22: note: ‘buckets’ declared here
DviCharNameHash *buckets[DVI_HASH_SIZE];
^~~~~~~
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Clears 42 out of 66 -Wdiscarded-qualifiers warnings from gcc
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
The information previously listed here didn't match what is present in
the source code or the COPYING file, and the X(7) man page doesn't list
any license information as this had claimed.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Found by using:
codespell --builtin clear,rare,usage,informal,code,names
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
This is the second version of patch.
It now uses fstat instead of stat as recommended in
review comments from:
Philipp Hagemeister <phihag@phihag.de>.
Bug was reported by:
Krzysztof Żelechowski <giecrilj@stegny.2a.pl>
Signed-off-by: Pat Kane <pekane52@gmail.com>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Mihail Konev <k.mvc@ya.ru>
|
|
Place quotes around the $srcdir, $ORIGDIR and $0 variables to prevent
fall-outs, when they contain space.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Syncs the invocation of configure with the one from the server.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
See http://people.gnome.org/~walters/docs/build-api.txt
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Regroup AC statements under the Autoconf initialization section.
Regroup AM statements under the Automake initialization section.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Found by parfait 1.1 bug checking tool:
Error: File Leak
File Descriptor Leak: Leaked File Descriptor fd
at line 457 of app/xditview/Dvi.c in function 'OpenFile'.
fd initialized at line 449 with mkstemp
Also reorganize & clean up the code a bit while we're here
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|