Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

This wasn't actually defined on Solaris, so I don't know what this was for.

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>

Since the FS_OPEN_MAX value is only used for calculating a hardcoded
array size, we can't really use a sysconf call to find it, so stop
pretending that's a valid answer that we'll just ignore later.

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>

AC_PROG_LIBTOOL was replaced by LT_INIT in libtool 2 in 2008,
so it's time to rely on it.

Clears autoconf warnings:

configure.ac:39: warning: The macro `AC_PROG_LIBTOOL' is obsolete.
configure.ac:39: You should run autoupdate.
aclocal.m4:3640: AC_PROG_LIBTOOL is expanded from...
configure.ac:39: the top level

libtoolize: Consider adding 'AC_CONFIG_MACRO_DIRS([m4])' to configure.ac,
libtoolize: and rerunning libtoolize and aclocal.

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Oracle no longer includes this term in our copyright & license notices.

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Needed for builds on NetBSD to work correctly, since it depends on
AC_USE_SYSTEM_EXTENSIONS defining _OPENBSD_SOURCE to expose the
prototype for reallocarray() in the system headers.

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Provides automatic integer overflow checking in allocation size calculations
(Though we already have explicit checks before using values provided
 over the protocol in calculations, so many were already manually checked.)

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Wrapper for realloc() that checks for overflow when multiplying
arguments together, so we don't have to add overflow checks to
every single call.  For documentation on usage, see:
http://www.openbsd.org/cgi-bin/man.cgi/OpenBSD-current/man3/calloc.3



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>

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Also clears up all "Loss of precision on implicit conversion" warnings
from Oracle's Parfait static analyser.

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

FSQXInfo.c: In function ‘FSQueryXInfo’:
FSQXInfo.c:110:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     for (j=0; j<props->num_offsets; j++)
                ^

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

FSQXExt.c: In function ‘FSQueryXExtents8’:
FSQXExt.c:105:19: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     for (i = 0; i < reply.num_extents; i++) {
                   ^
FSQXExt.c: In function ‘FSQueryXExtents16’:
FSQXExt.c:141:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
  for (i = 0; i < str_len; i++) {
                ^
FSQXExt.c:165:19: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     for (i = 0; i < reply.num_extents; i++) {
                   ^

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

FSQGlyphs.c: In function ‘FSQueryXBitmaps8’:
FSQGlyphs.c:113:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     for (i=0; i<reply.num_chars; i++)
                ^
FSQGlyphs.c: In function ‘FSQueryXBitmaps16’:
FSQGlyphs.c:160:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
  for (i = 0; i < str_len; i++) {
                ^
FSQGlyphs.c:198:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     for (i=0; i<reply.num_chars; i++)
                ^

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

FSListCats.c: In function ‘FSListCatalogues’:
FSListCats.c:104:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
  for (i = 0; i < rep.num_catalogues; i++) {
                ^

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

FSFontInfo.c: In function ‘FSListFontsWithXInfo’:
FSFontInfo.c:182:13: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
  for (j=0; j<pi[i]->num_offsets; j++)
             ^

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Mihail Konev authored 8 years ago and Peter Hutterer committed 8 years ago

Signed-off-by: Mihail Konev <k.mvc@ya.ru>

Emil Velikov authored 9 years ago and Peter Hutterer committed 8 years ago

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>

Parts were indented, others weren't, now is more consistent.
'git diff -w' shows no non-whitespace changes in this commit

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>

The contents of pattern are simply copied to the request sent to
the font server and never touched again, so can be labeled read-only.

Fixes compiler warning building xfsinfo:

xfsinfo.c: In function ‘print_catalogue_info’:
xfsinfo.c:158:5: warning: passing argument 2 of ‘FSListCatalogues’ discards
 ‘const’ qualifier from pointer target type [enabled by default]
     char      **cats = FSListCatalogues(svr, "*", 1000, &n);
     ^
In file included from xfsinfo.c:66:0:
FSlib.h:255:16: note: expected ‘char *’ but argument is of type ‘const char *’
 extern char ** FSListCatalogues ( FSServer *svr, char *pattern,

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>