Create a default resolve implementation in Type that
simply sets resolved = True, and thus we can skip implementing
the resolve method when theres nothing to resolve.
Instead of naming the parent class explicitly when calling the
parent-class's resolve, use super()
Signed-off-by: Robert Knutsson <zybreak@gmail.com>
Instead of an implicit dependency on __main__.output at
import time, instead use the output dict already passed
to the Module class's constructor.
This also removes the awkward requirement to define the output dict
before importing xcbgen
Signed-off-by: Robert Knutsson <zybreak@gmail.com>
Use relative imports within the package.
Only import exactly whats needed in order not to accidentally export
internal symbols.
Explicitly state whats part of the module when doing a wildcard import.
Signed-off-by: Robert Knutsson <zybreak@gmail.com>
Convert Python2 new-style class definitions Foo(object) to class Foo and replace explicit
BaseClass.__init__() calls with super().__init__() for cleaner,
modern Python 3 code.
Signed-off-by: Robert Knutsson <zybreak@gmail.com>
Replies are always 32 bytes, so many messages have padding at the end of them.
The XCB code doesn't have a problem with the missing padding fields, but they
are nice to have so they match the spec and other users of this data don't have
to check message lengths.
Two cases had padding but with shorter amounts than required.
Part-of: <https://gitlab.freedesktop.org/xorg/proto/xcbproto/-/merge_requests/60>
Performed with: git ls-files | xargs perl -i -p -e 's{[ \t]+$}{}'
git diff -w & git diff -b show no diffs from this change
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Instead of using "Left Double Quotation Mark", this replaces some
documentation comments with normal, ASCII-compatible quotation marks.
Related-to: https://gitlab.freedesktop.org/xorg/lib/libxcb/-/issues/72
Signed-off-by: Uli Schlachter <psychon@znc.in>
Otherwise it's hard to tell where this came from, and we qualify most
everything else in this file anyway.
Signed-off-by: Tycho Andersen <tycho@tycho.pizza>
This is useful, but it crashes the XCB code generator.
xorg/lib/libxcb!38 prevents the crash.
Signed-off-by: Demi Marie Obenour <demiobenour@gmail.com>
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>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
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>
configure.ac:22: warning: AC_OUTPUT should be used without arguments.
configure.ac:22: You should run autoupdate.
Signed-off-by: Matt Turner <mattst88@gmail.com>
Added by 17ab095 ("add libdir/exec_prefix to xcb-xproto.pc") with the
rationale of using it to find the python path, but later commits bd7a481
("pkg-config: Add sysroot prefix") and be9c255 ("Adjust to automake
1.16.4 changes") make this directly accessible.
By removing libdir from the pkgconfig file, the file becomes identical
for different ABIs.
Signed-off-by: Matt Turner <mattst88@gmail.com>
Adjacent lists of char should not have any automatic padding added,
since the alignment requirements for both are the same. Add explicit
padding where it is required.
Signed-off-by: Peter Harris <pharris@opentext.com>
