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

As suggested by clang:
xlsatoms.c:56:1: warning: function 'usage' could be declared with
 attribute 'noreturn' [-Wmissing-noreturn]
{
^

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

xlsatoms.c:175:13: warning: implicit conversion loses integer precision:
  'long long' to 'xcb_atom_t' (aka 'unsigned int') [-Wshorten-64-to-32]
    *atom = value;
          ~ ^~~~~

xlsatoms.c:244:37: warning: implicit conversion loses integer precision:
  'long' to 'xcb_atom_t' (aka 'unsigned int') [-Wshorten-64-to-32]
        cookie[i] = xcb_get_atom_name(c, i + low);
                    ~~~~~~~~~~~~~~~~~    ~~^~~~~

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>

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

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

Tobias Stoeckmann authored Jul 04, 2018 and Alan Coopersmith committed Nov 12, 2018

The default format is %lu\t%s, not %ld\t%s, i.e. unsigned.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Tobias Stoeckmann authored Jul 04, 2018 and Alan Coopersmith committed Nov 12, 2018

If a low and high range limit has been specified, all atoms are
retrieved at once. This is also the reason why malloc() is used:
All cookies are stored before collecting the data.

By using chunks it is possible to specify a huge range or even
all possible atoms without running out of memory.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Tobias Stoeckmann authored Jul 04, 2018 and Alan Coopersmith committed Nov 12, 2018

The manual page states that if no upper range limit has been specified,
no higher atoms will be printed. This is not true for

$ xlsatoms -range 0-

This prints the first 100 atoms, even though it already encountered an
invalid one at 0. The reason is that say_batch works as a batch, i.e.
retrieves 100 atoms at a time. If one of them is invalid, the rest is
still printed.

With this adjustment, xlsatoms behaves as stated in manual page.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Tobias Stoeckmann authored Jul 04, 2018 and Alan Coopersmith committed Nov 12, 2018

The data type xcb_atom_t is an unsigned int (32 bit), but the optional
range argument is parsed with atoi(), which returns a signed int.

Even though it is possible to reach all values through clever casting,
it is more readable by properly using correct data types.

This also fixes a segmentation fault on 32 bit systems if a range is
supplied which overflows size_t:

$ xlsatoms -range 0-1073741824
Segmentation fault (core dumped)

If an invalid range is supplied, an error message is printed. This is
new because previously an invalid range was silently accepted.

$ xlsatoms -range 0--1
$ _
$ xlsatoms-new -range 0--1
xlsatoms-new:  invalid range: 0--1

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Mihail Konev authored Jan 26, 2017 and Peter Hutterer committed Jan 26, 2017

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

Emil Velikov authored Mar 09, 2015 and Peter Hutterer committed Jan 26, 2017

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>

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>

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

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

Use services provided by XORG_MANPAGE_SECTIONS.
Use standard Makefile for man pages.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

Using s/[ \t]*$//

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

This silences an Autoconf warning

This macro is called by PKG_CHECK_MODULES

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

This macro is called by PKG_CHECK_MODULES

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

XORG_STRICT_OPTION from XORG_DEFAULT_OPTIONS calls
AC_PROG_C_C99. This sets gcc with -std=gnu99.
If AC_PROG_CC macro is called afterwards, it resets CC to gcc.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

It depends on util-macros 1.8 or later
The existing statement can now be removed from the configuration file.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

Use MAN_SUBST now supplied in XORG_MANPAGE_SECTIONS
The value of MAN_SUBST is the same for all X.Org packages.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

Unrelated to the previous patches, the new value simply reflects
the reality that the minimum level for autoconf to configure
all x.org modules in one pass is 2.60 dated June 2006.

A version later than 2.60 can be used, but no new features from such
a later version can be used in configure.ac.

Although the code in this module can be configured with a version earlier
than 2.60, most of code is now contained in macros from util-macros
which use features of version 2.60, at the present or in the future.

ftp://ftp.gnu.org/gnu/autoconf/autoconf-2.60.tar.gz



Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

Jesse Adkins authored Sep 28, 2010 and Alan Coopersmith committed Oct 06, 2010

Signed-off-by: Jesse Adkins <jesserayadkins@gmail.com>

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

This dramatically improves latency, at the cost of a small amount of
bandwidth.

(Theoretically. In practice, it actually saves bandwidth by reducing the
number of IP headers, Ethernet frames, PPP headers, ATM frames...)

Signed-off-by: Peter Harris <pharris@opentext.com>
Reviewed-By: Jeremy Huddleston <jeremyhu@apple.com>

Now that the INSTALL file is generated.
Allows running make maintainer-clean.