Enrico Weigelt, metux IT consult
ba665948f6
os: unexport WriteFdToClient()
...
Not used by any modules, so no need to keep it exported.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790 >
2025-02-18 10:53:44 +00:00
Enrico Weigelt, metux IT consult
effe8d66bd
os: unexport ReadRequestFromClient()
...
Not used by any modules, so no need to keep it exported.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790 >
2025-02-18 10:53:44 +00:00
Enrico Weigelt, metux IT consult
9b0c3c62f2
os: unexport ListenOnOpenFD()
...
Not used by any modules, so no need to keep it exported.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790 >
2025-02-18 10:53:44 +00:00
Enrico Weigelt, metux IT consult
d637e1d284
os: unexport AddClientOnOpenFD()
...
Not used by any modules, so no need to keep it exported.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790 >
2025-02-18 10:53:44 +00:00
Enrico Weigelt, metux IT consult
a84ae08526
os: unexport ClientAuthorized()
...
Not used by any modules, so no need to keep it exported.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790 >
2025-02-18 10:53:44 +00:00
Enrico Weigelt, metux IT consult
c6b8207342
os: unexport AuthorizationIDOfClient()
...
Not used by any modules, so no need to keep it exported.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790 >
2025-02-18 10:53:44 +00:00
Enrico Weigelt, metux IT consult
1f27d3d653
os: unexport ClientIsLocal()
...
Not used by any modules, so no need to keep it exported.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790 >
2025-02-18 10:53:44 +00:00
Enrico Weigelt, metux IT consult
9b3b938ede
os: re-add System() function for backwards compat
...
This function had been unexported long ago, but the Intel driver
still needs it. Adding a tiny temporary compat wrapper, so Intel
team has time to keep up with us.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1778 >
2025-02-11 09:53:59 +01:00
Enrico Weigelt, metux IT consult
f4eb217f11
os: log: replace VErrorF() by LogVMessageVerb()
...
Since LogVMessageVerb() is now signal safe, we can use this one instead.
Leaving VErrorF() macro for backwards compat with drivers.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1691 >
2025-02-06 23:35:27 +00:00
Enrico Weigelt, metux IT consult
da22bc9ae0
os: log: replace VErrorFSigSafe() by VErrorF()
...
Since VErrorF() is now signal safe, we can use this one instead.
Leaving VErrorFSigSafe() macro for backwards compat with drivers.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1691 >
2025-02-06 23:35:27 +00:00
Enrico Weigelt, metux IT consult
2d18c353b4
os: log: replace ErrorFSigSafe() by ErrorF()
...
Since ErrorF() is now signal safe, we can use this one instead.
Leaving ErrorFSigSafe() macro for backwards compat with drivers.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1691 >
2025-02-06 23:35:27 +00:00
Enrico Weigelt, metux IT consult
dd37cc4855
os: log: replace LogMessageVerbSigSafe() by LogMessageVerb()
...
Since LogMessageVerb() is now signal safe, we can use this one instead.
Leaving LogMessageVerbSigSafe() macro for backwards compat with drivers.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1691 >
2025-02-06 23:35:27 +00:00
Enrico Weigelt, metux IT consult
5962211612
os: log: make LogVMessageVerb() signal safe
...
We already have our own signal safe vnsprintf() implementation, which is used
for formatting log messages while being in a signal handler, there's no need
to have two separate implementations of all the logging functions.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1691 >
2025-02-06 23:35:27 +00:00
Enrico Weigelt, metux IT consult
161c3c8398
os: drop obsolete LogHdrMessage()
...
This function isn't used anywhere, so no need to keep it around.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1677 >
2025-02-06 19:57:52 +00:00
Enrico Weigelt, metux IT consult
e5b5b302d3
os: replace LogVWrite() by LogVMessageVerb()
...
It's just a wrapper around LogVMessageVerb() and no external module
using it, so can easily be optimized-away.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1683 >
2024-10-10 17:34:12 +00:00
Enrico Weigelt, metux IT consult
c26023a009
os: replace LogWrite() by LogMessageVerb()
...
It's nothing but a wrapper, doing the same as LogMessageVerb(X_NONE, ...),
and no external module / driver needs it, so can be easily optimized away.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1683 >
2024-10-10 17:34:12 +00:00
Enrico Weigelt, metux IT consult
37446b70de
os.h: drop unnecessary guard on stdlib.h include
...
It's a standard C header, that's always present, so no extra guard needed.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1684 >
2024-10-10 17:16:00 +00:00
Enrico Weigelt, metux IT consult
1d7cb4b2c3
os: unexport CloseDownConnection()
...
Not used by any drivers, so no need to export it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1582 >
2024-09-02 16:02:39 +00:00
Enrico Weigelt, metux IT consult
811977228b
os: unexport screen saver timer functions
...
These functions aren't supposed to be used by drivers, so move them
out of the public API.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1583 >
2024-09-01 20:23:10 +00:00
Enrico Weigelt, metux IT consult
3d3d148039
os: unexport WaitForSomething()
...
Not used by any drivers, so no need to export it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1584 >
2024-09-01 19:53:43 +00:00
Enrico Weigelt, metux IT consult
61233adbca
treewide: replace xnfreallocarray macro call by XNFreallocarray()
...
The xnfreallocarray was added along (and just as an alias to) XNFreallocarray
back a decade ago. It's just used in a few places and it's only saves us from
passing the first parameter (NULL), so the actual benefit isn't really huge.
No (known) driver is using it, so the macro can be dropped entirely.
Fixes: ae75d50395info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1529 >
2024-07-26 23:41:33 +00:00
Enrico Weigelt, metux IT consult
6b3c916030
treewide: replace xnfallocarray() calls by XNFreallocarray
...
The xnfallocarray was added along (and just as an alias to) XNFreallocarray
back a decade ago.
No (known) driver is using it, so the macro can be dropped entirely.
Fixes: ae75d50395info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1529 >
2024-07-26 23:41:33 +00:00
Enrico Weigelt, metux IT consult
4773182c72
os: unexport LockServer() and UnlockServer() from public module API
...
These functions work on server process level, and shouldn't be touched
by drivers at all, thus shouldn't be exported to them.
(couldn't find any driver/module using these symbols)
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1320 >
2024-07-26 22:26:07 +00:00
Enrico Weigelt, metux IT consult
b30edf326b
fix missing includes of <X11/Xfuncproto.h>
...
Several places using _X_ATTRIBUTE_PRINTF macro from X11/Xfuncproto.h
but missing to include it, so it depends on other headers whether it's
included by mere accident, which quickly causes trouble if include order
changes. Cleaning that up by adding explicit include statements.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1580 >
2024-07-20 17:18:38 +00:00
Enrico Weigelt, metux IT consult
e5c8b664d3
os: unexport ddx callbacks
...
The DDX callbacks (where core/DIX calls into DDX) aren't supposed to be
called by drivers directly, so unexport them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1332 >
2024-05-08 09:37:35 +02:00
Enrico Weigelt, metux IT consult
c17c527b9a
os: unexport ForceClockId()
...
This function is only intended for DDX'es (currently just Xwayland) that
need to force using another clock. Really shouldn't be called by drivers
or extensions.
Fixes: a779fda224info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1333 >
2024-04-17 16:20:54 -07:00
Enrico Weigelt, metux IT consult
a0b69624f7
os: unexport ListenToAllClients()
...
It's just called by DIX, not by any drivers/modules, so no need to export it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1387 >
2024-04-16 18:23:27 -07:00
Enrico Weigelt, metux IT consult
d18ef33099
os: unexport OnlyListenToOneClient()
...
Not used by any drivers, so no need to export it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1385 >
2024-04-16 18:13:11 -07:00
Enrico Weigelt, metux IT consult
4c1795701c
os: unexport MakeClientGrabPervious() and MakeClientGrabImpervious()
...
Not used by any drivers, so no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1384 >
2024-04-16 17:52:14 -07:00
Enrico Weigelt, metux IT consult
be4c8444eb
os: unexport Os*() functions
...
These aren't called (and suited for being called) by drivers,
thus drop them from the public module API.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1381 >
2024-04-16 14:20:30 +02:00
Enrico Weigelt, metux IT consult
76d01e9bf6
os: unexport OsVendorVErrorFProc pointer
...
This pointer allows a DDX to install it's own error print handler. It's really
only intended for DDXes, thus no need to have it exported to modules.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1369 >
2024-04-15 23:59:38 +00:00
Enrico Weigelt, metux IT consult
6cb599f59c
os: unexport OsLookupColor()
...
It's an internal function, only used by DIX, not by drivers and shouldn't
have been exported in the first place.
Fixes: 49f77fff14info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1373 >
2024-04-15 16:19:06 -07:00
Enrico Weigelt, metux IT consult
3b7a63e6ae
os: unexport xthread_sigmask
...
This function isn't used by drivers and there's currently no need to do so,
thus keep it out of the public module API.
Fixes: 30ac756798info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1368 >
2024-04-15 22:49:02 +00:00
Enrico Weigelt, metux IT consult
cf5fab0bbe
os: unexport AccessUsingXdmcp()
...
This function is only used internally for xdmcp auth handling, not used
in modules/drivers, so no need to export it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:55:22 -07:00
Enrico Weigelt, metux IT consult
dd21a80219
os: unexport ChangeAccessControl()
...
this function isn't used by modules/drivers, so no need to export it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:55:22 -07:00
Enrico Weigelt, metux IT consult
3f00c1b6cf
os: unexport host control functions
...
these aren't used by modules/drivers, so no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:55:22 -07:00
Enrico Weigelt, metux IT consult
5a9b885118
os: unexport authorization management functions
...
Those aren't used by modules, thus no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:55:22 -07:00
Enrico Weigelt, metux IT consult
bed778ee60
os: unexport LocalAccessScopeUser()
...
this function is only used by wayland ddx, so no need to export it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:55:22 -07:00
Enrico Weigelt, metux IT consult
e621081fa3
os: unexport EnableLocalUser() and DisableLocalUser()
...
these aren't used by module, so no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:55:22 -07:00
Enrico Weigelt, metux IT consult
54441ff98a
os: unexport EnableLocalAccess() and DisableLocalAccess()
...
these aren't used by modules, so no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:44:36 -07:00
Enrico Weigelt, metux IT consult
b00fdd482c
os: unexport EnableLocalHost() and DisableLocalHost()
...
These aren't used by modules, so no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:44:36 -07:00
Enrico Weigelt, metux IT consult
5f19eab1ee
os: unexport local client creds handling
...
these functions aren't used by modules, so no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:44:34 -07:00
Enrico Weigelt, metux IT consult
cbc9f557e5
os: unexport set_font_authorizations()
...
This function is only used by DIX, so no need to export it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:42:45 -07:00
Enrico Weigelt, metux IT consult
389b528203
os: unexport command line args handling functions
...
These functions shouldn't be called by drivers or extensions, thus
shouldn't be exported. Also moving it to separate header, so the
already huge ones aren't cluttered with even more things.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1334 >
2024-04-15 21:21:40 +00:00
Enrico Weigelt, metux IT consult
3843a643e0
os: unexport audit functions
...
These aren't used by any drivers, so no need to keep them around in the
public driver API.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1326 >
2024-04-15 19:54:34 +00:00
Enrico Weigelt, metux IT consult
30b403b0f0
os: unexport Fopen(), Fclose(), Popen(), Pclose(), System()
...
These functions are just used for reading auth file or calling xkbcomp while
dropping privileges, in case the Xserver is started as unprivileged user
with suid-root. Thus, shouldn't be used (and aren't used) by drivers.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1370 >
2024-03-09 17:18:46 +00:00
Enrico Weigelt, metux IT consult
15d3c1a6f1
os: move os_move_fd() out of public API
...
This function isn't used by any driver and doesn't seem to be useful for them,
thus move it out of the public module API, in order to tidy it up a bit.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1366 >
2024-03-09 17:12:46 +00:00
Enrico Weigelt, metux IT consult
934b771f7a
os: unexport AutoResetServer()
...
This is an internal function, not used and not supposed to be used by
any drivers, so shouldn't have been exported in the first place.
Fixes: 49f77fff14info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1372 >
2024-03-07 23:10:17 +00:00
Enrico Weigelt, metux IT consult
c3255fbc1c
os: drop unneeded DDXOSVERRORF conditional
...
This conditional practically only controls whether we have an pointer,
where DDX can plug in it's own VErrorF() handler (currently only xwin
doing that). The cost of having it even when DDX doesn't use it, is
really negligible: it's just one pointer and an extra non-null check
on it per VErrorF() call - a very cold path. Strangely, xwin has extra
Getting rid of this unnecessary complexity that really hasn't any
practical gain.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1329 >
2024-03-03 23:15:14 +00:00
Enrico Weigelt, metux IT consult
5d2f675ca8
os: utils: drop obsolete System() on non-win32 targets
...
This function is only used on WIN32 targets, so we can drop the *nix
implementation.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net >
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1295 >
2024-03-03 23:02:54 +00:00
