<feed xmlns='http://www.w3.org/2005/Atom'>
<title>wayland/tests/queue-test.c, branch 1.23</title>
<subtitle>[no description]</subtitle>
<id>http://git.sudomsg.com/mirror/wayland/atom/tests/queue-test.c?h=1.23</id>
<link rel='self' href='http://git.sudomsg.com/mirror/wayland/atom/tests/queue-test.c?h=1.23'/>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/'/>
<updated>2024-01-22T12:34:14Z</updated>
<entry>
<title>client: Allow setting names for queues</title>
<updated>2024-01-22T12:34:14Z</updated>
<author>
<name>Derek Foreman</name>
<email>derek.foreman@collabora.com</email>
</author>
<published>2023-11-10T20:21:48Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=b42218f790033c496c48c7d5ba1f8421db794d0b'/>
<id>urn:sha1:b42218f790033c496c48c7d5ba1f8421db794d0b</id>
<content type='text'>
Allow setting a name for an event queue. The queue is used only for
printing additional debug information.

Debug output can now show the name of the event queue an event is
dispatched from, or the event queue of a proxy when a request is made.

Signed-off-by: Derek Foreman &lt;derek.foreman@collabora.com&gt;
</content>
</entry>
<entry>
<title>debug: Replace "@&lt;id&gt;" with "#&lt;id&gt;" in logs</title>
<updated>2023-06-27T21:08:25Z</updated>
<author>
<name>Alex Yang</name>
<email>aycyang@google.com</email>
</author>
<published>2023-05-24T18:24:23Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=6d3334657180f3663b8d83636f4d1bc2097492d2'/>
<id>urn:sha1:6d3334657180f3663b8d83636f4d1bc2097492d2</id>
<content type='text'>
Wayland debug logs resemble email addresses. This is a problem when
anonymizing logs from users. For example:

[2512874.343] xdg_surface@700.configure(333)

In the above log line, the substring "surface@700.config" can be
mistaken for an email address and redacted during anonymization.

Signed-off-by: Alex Yang &lt;aycyang@google.com&gt;
</content>
</entry>
<entry>
<title>client: Do not warn about attached proxies on default queue destruction.</title>
<updated>2023-03-01T13:50:02Z</updated>
<author>
<name>Alexandros Frantzis</name>
<email>alexandros.frantzis@collabora.com</email>
</author>
<published>2023-02-28T21:42:59Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=b01a85dfd5e8cda7170b7ba6fe66fc3800f93990'/>
<id>urn:sha1:b01a85dfd5e8cda7170b7ba6fe66fc3800f93990</id>
<content type='text'>
If the default queue is being destroyed, the client is disconnecting
from the wl_display, so there is no possibility of subsequent events
being queued to the destroyed default queue, which is what this warning
is about.

Note that interacting with (e.g., destroying) a wl_proxy after its
wl_display is destroyed is a certain memory error, and this warning will
indirectly warn about this issue. However, this memory error should be
detected and warned about through a more deliberate mechanism.

Signed-off-by: Alexandros Frantzis &lt;alexandros.frantzis@collabora.com&gt;
</content>
</entry>
<entry>
<title>client: Abort when trying to add an event to a destroyed queue</title>
<updated>2023-02-28T11:22:04Z</updated>
<author>
<name>Alexandros Frantzis</name>
<email>alexandros.frantzis@collabora.com</email>
</author>
<published>2022-11-15T09:44:55Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=d72f9007c36f2f8ad2dc26178545e8a7f5b993a0'/>
<id>urn:sha1:d72f9007c36f2f8ad2dc26178545e8a7f5b993a0</id>
<content type='text'>
Detect when we are trying to add an event to a destroyed queue,
and abort instead of causing a use-after-free memory error.

This situation can occur when an wl_event_queue is destroyed before
its attached wl_proxy objects.

Signed-off-by: Alexandros Frantzis &lt;alexandros.frantzis@collabora.com&gt;
</content>
</entry>
<entry>
<title>client: Warn when a queue is destroyed with attached proxies</title>
<updated>2023-02-28T11:22:04Z</updated>
<author>
<name>Alexandros Frantzis</name>
<email>alexandros.frantzis@collabora.com</email>
</author>
<published>2022-11-15T08:50:53Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=0ba650202e742b23150054cf0740168cd529b010'/>
<id>urn:sha1:0ba650202e742b23150054cf0740168cd529b010</id>
<content type='text'>
Log a warning if the queue is destroyed while proxies are still
attached, to help developers debug and fix potential memory errors.

Signed-off-by: Alexandros Frantzis &lt;alexandros.frantzis@collabora.com&gt;
</content>
</entry>
<entry>
<title>(multiple): Include stdint.h</title>
<updated>2016-07-26T01:39:32Z</updated>
<author>
<name>Yong Bakos</name>
<email>ybakos@humanoriented.com</email>
</author>
<published>2016-07-18T17:42:25Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=2b1c1b2d66945c5cb237a7264759b6dbdf42b02a'/>
<id>urn:sha1:2b1c1b2d66945c5cb237a7264759b6dbdf42b02a</id>
<content type='text'>
Some headers and source files have been using types such as uint32_t
without explicitly including stdint.h.

Explicitly include stdint.h where appropriate.

Signed-off-by: Yong Bakos &lt;ybakos@humanoriented.com&gt;
Reviewed-by: Eric Engestrom &lt;eric.engestrom@imgtec.com&gt;
</content>
</entry>
<entry>
<title>tests/queue-test: Add tests for proxy wrappers</title>
<updated>2016-04-29T12:58:23Z</updated>
<author>
<name>Jonas Ådahl</name>
<email>jadahl@gmail.com</email>
</author>
<published>2016-04-29T11:30:23Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=69ec70fb0d3f75f4bcce449238d6297f6a986b5f'/>
<id>urn:sha1:69ec70fb0d3f75f4bcce449238d6297f6a986b5f</id>
<content type='text'>
Test that doing wl_display.sync on a wrapped proxy with a special queue
works as expected.

Test that creating a wrapper on a destroyed but not freed proxy fails.

Signed-off-by: Jonas Ådahl &lt;jadahl@gmail.com&gt;
Reviewed-by: Pekka Paalanen &lt;pekka.paalanen@collabora.co.uk&gt;
</content>
</entry>
<entry>
<title>tests: Pass argument to client main</title>
<updated>2016-01-16T08:37:37Z</updated>
<author>
<name>Jonas Ådahl</name>
<email>jadahl@gmail.com</email>
</author>
<published>2015-12-28T03:25:00Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=046012a6cf1a50e1de2c73cf5a29098453b0171f'/>
<id>urn:sha1:046012a6cf1a50e1de2c73cf5a29098453b0171f</id>
<content type='text'>
Change the API to pass an "void *" argument to the client main
function, allowing the caller to call the same main function with
different input.

A helper (client_create_noarg) is added for when no argument is passed,
and the existing test cases are changed to use this function instead.

Signed-off-by: Jonas Ådahl &lt;jadahl@gmail.com&gt;
Reviewed-by: Daniel Stone &lt;daniels@collabora.com&gt;
</content>
</entry>
<entry>
<title>tests: Update boilerplate from MIT X11 license to MIT Expat license</title>
<updated>2015-06-12T22:31:24Z</updated>
<author>
<name>Bryce Harrington</name>
<email>bryce@osg.samsung.com</email>
</author>
<published>2015-06-10T09:57:59Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=773babedfca8f0aa60903aed4a6856cd7e36227d'/>
<id>urn:sha1:773babedfca8f0aa60903aed4a6856cd7e36227d</id>
<content type='text'>
Signed-off-by: Bryce Harrington &lt;bryce@osg.samsung.com&gt;
Reviewed-by: Pekka Paalanen &lt;pekka.paalanen@collabora.co.uk&gt;
</content>
</entry>
<entry>
<title>tests: split queue-test testcases</title>
<updated>2014-12-01T10:04:18Z</updated>
<author>
<name>Marek Chalupa</name>
<email>mchqwerty@gmail.com</email>
</author>
<published>2014-11-21T10:15:24Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=fdb519be08b35c7d9659b0476aec78c170f7680d'/>
<id>urn:sha1:fdb519be08b35c7d9659b0476aec78c170f7680d</id>
<content type='text'>
All the test-cases are in one test atm. It doesn't matter for the
outcome, but when it is split to more tests, the debugging and reading
the output is simpler.

Signed-off-by: Marek Chalupa &lt;mchqwerty@gmail.com&gt;
Acked-by: Pekka Paalanen &lt;pekka.paalanen@collabora.co.uk&gt;
</content>
</entry>
<entry>
<title>queue-test: put back timeout</title>
<updated>2014-11-17T14:59:50Z</updated>
<author>
<name>Marek Chalupa</name>
<email>mchqwerty@gmail.com</email>
</author>
<published>2014-09-24T12:37:08Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=cf11e9dfbe41d84d537ca11453abbca17170d146'/>
<id>urn:sha1:cf11e9dfbe41d84d537ca11453abbca17170d146</id>
<content type='text'>
In 93e654061b9 we removed call to alarm() that served as timeout in this test.
Now when we have test_set_timeout() func, return the timeout back.

Signed-off-by: Marek Chalupa &lt;mchqwerty@gmail.com&gt;
Reviewed-by: Giulio Camuffo &lt;giuliocamuffo@gmail.com&gt;
</content>
</entry>
<entry>
<title>queue-test: Add another assertion</title>
<updated>2014-11-05T12:26:53Z</updated>
<author>
<name>Philip Withnall</name>
<email>philip@tecnocode.co.uk</email>
</author>
<published>2014-11-01T17:06:46Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=2097414a7c8febf89305e1945f51d7fa9972c41f'/>
<id>urn:sha1:2097414a7c8febf89305e1945f51d7fa9972c41f</id>
<content type='text'>
Ensure that the round trip succeeds.

Signed-off-by: Philip Withnall &lt;philip at tecnocode.co.uk&gt;
Signed-off-by: Karsten Otto &lt;ottoka at posteo.de&gt;
Reviewed-by: Marek Chalupa &lt;mchqwerty@gmail.com&gt;
Reviewed-by: Pekka Paalanen &lt;pekka.paalanen@collabora.co.uk&gt;
</content>
</entry>
<entry>
<title>tests: use test compositor in queue-test</title>
<updated>2014-08-22T09:39:52Z</updated>
<author>
<name>Marek Chalupa</name>
<email>mchqwerty@gmail.com</email>
</author>
<published>2014-08-21T14:53:14Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=93e654061b9497c78b6f89a0b7f54107029e2628'/>
<id>urn:sha1:93e654061b9497c78b6f89a0b7f54107029e2628</id>
<content type='text'>
Most of the code of the queue-test is covered by the test compositor,
so we can save few lines and use the test compositor instead.
I think it's also more readable.

This patch removes timeout from the test. We plan to add timeout
to all tests later, though.

v2.
  rebased to master

Signed-off-by: Marek Chalupa &lt;mchqwerty@gmail.com&gt;
Reviewed-by: Pekka Paalanen &lt;pekka.paalanen@collabora.co.uk&gt;
</content>
</entry>
<entry>
<title>tests: remove unnecessary lines from queue-test</title>
<updated>2014-08-21T11:44:51Z</updated>
<author>
<name>Marek Chalupa</name>
<email>mchqwerty@gmail.com</email>
</author>
<published>2014-08-21T10:07:07Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=c6d98e15cae20b481f954ce50f788644d1286b62'/>
<id>urn:sha1:c6d98e15cae20b481f954ce50f788644d1286b62</id>
<content type='text'>
Earlier, the wl_display_dispatch_pending were setting number of thread
that can dispatch events. This behaviour was removed later,
so now these lines are redundant.

Related commits:

385fe30e8b144a968aa88c6546c2ef247771b3d7
78cfa967681c965d23f6cbf76e080bbb0b564ff6
3c7e8bfbb4745315b7bcbf69fa746c3d6718c305

Signed-off-by: Marek Chalupa &lt;mchqwerty@gmail.com&gt;
Acked-by: Pekka Paalanen &lt;pekka.paalanen@collabora.co.uk&gt;
</content>
</entry>
<entry>
<title>tests: remove leaks from queue-test</title>
<updated>2014-08-21T11:44:39Z</updated>
<author>
<name>Marek Chalupa</name>
<email>mchqwerty@gmail.com</email>
</author>
<published>2014-08-21T10:07:06Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=8cf7a23e57495f079c9b2b773ac36646a47905d6'/>
<id>urn:sha1:8cf7a23e57495f079c9b2b773ac36646a47905d6</id>
<content type='text'>
Destroy all objects that we have created

Signed-off-by: Marek Chalupa &lt;mchqwerty@gmail.com&gt;
Reviewed-by: Pekka Paalanen &lt;pekka.paalanen@collabora.co.uk&gt;
</content>
</entry>
<entry>
<title>tests: test the wl_display_roundtrip_queue() function</title>
<updated>2014-08-21T07:30:26Z</updated>
<author>
<name>Giulio Camuffo</name>
<email>giuliocamuffo@gmail.com</email>
</author>
<published>2014-08-20T16:29:10Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=a52357f6fbfb5f531e91dc2eb85c93c86c2a628f'/>
<id>urn:sha1:a52357f6fbfb5f531e91dc2eb85c93c86c2a628f</id>
<content type='text'>
[Pekka Paalanen: moved variable declarations to before code. Added some
comments, and added the re-arm to additionally test the opposite case.]

Signed-off-by: Pekka Paalanen &lt;pekka.paalanen@collabora.co.uk&gt;
</content>
</entry>
<entry>
<title>queue-test: assert non-NULL return values</title>
<updated>2014-01-15T18:46:08Z</updated>
<author>
<name>U. Artie Eoff</name>
<email>ullysses.a.eoff@intel.com</email>
</author>
<published>2014-01-10T21:18:42Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=22a4a95873a4ccfc1c986761782b666442dd244a'/>
<id>urn:sha1:22a4a95873a4ccfc1c986761782b666442dd244a</id>
<content type='text'>
Signed-off-by: U. Artie Eoff &lt;ullysses.a.eoff@intel.com&gt;
</content>
</entry>
<entry>
<title>Replace two remaining wl_display_add_gloavl() occurences</title>
<updated>2013-07-09T23:18:10Z</updated>
<author>
<name>Kristian Høgsberg</name>
<email>krh@bitplanet.net</email>
</author>
<published>2013-07-09T23:18:10Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=7100a5e0bb07d77f1700abe53968ffc5ba3749ce'/>
<id>urn:sha1:7100a5e0bb07d77f1700abe53968ffc5ba3749ce</id>
<content type='text'>
</content>
</entry>
<entry>
<title>queue-test: WEXITSTATUS() is undefined if WIFEXITED() is false</title>
<updated>2013-02-08T16:38:59Z</updated>
<author>
<name>Kristian Høgsberg</name>
<email>krh@bitplanet.net</email>
</author>
<published>2013-02-08T16:38:59Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=adcb2d73b3377c749efcae5ae0e5471e10b6646f'/>
<id>urn:sha1:adcb2d73b3377c749efcae5ae0e5471e10b6646f</id>
<content type='text'>
If a child process dies from a signal, WIFEXITED() returns false and
WEXITSTATUS() isn't well-defined.  In this case, if the client segfaults,
the status is 134 and WEXITSTATUS(134) is EXIT_SUCCESS, so we mask the error.
</content>
</entry>
<entry>
<title>tests: Add out of order delete_id queue tests</title>
<updated>2012-11-07T23:37:55Z</updated>
<author>
<name>Jonas Ådahl</name>
<email>jadahl@gmail.com</email>
</author>
<published>2012-11-06T07:15:04Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=2a8da76fb1b3afdb633a9551054ff9cecb89beca'/>
<id>urn:sha1:2a8da76fb1b3afdb633a9551054ff9cecb89beca</id>
<content type='text'>
Verify that when receiving the first of two synchronization callback
events, destroying the second one doesn't cause any errors even if the
delete_id event is handled out of order.

Signed-off-by: Jonas Ådahl &lt;jadahl@gmail.com&gt;
</content>
</entry>
<entry>
<title>tests: Use a pipe for synchronization to avoid potential deadlock</title>
<updated>2012-11-07T23:37:46Z</updated>
<author>
<name>Jonas Ådahl</name>
<email>jadahl@gmail.com</email>
</author>
<published>2012-11-06T07:15:03Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=3d651dc2eecbfba8c1fc813eedcf615364d5a2ac'/>
<id>urn:sha1:3d651dc2eecbfba8c1fc813eedcf615364d5a2ac</id>
<content type='text'>
Using signals in the previous way could potentially lead to dead locks
if the SIGCONT was signalled before a listener was registered.

Signed-off-by: Jonas Ådahl &lt;jadahl@gmail.com&gt;
</content>
</entry>
<entry>
<title>tests: Add queue test case</title>
<updated>2012-11-05T20:48:12Z</updated>
<author>
<name>Jonas Ådahl</name>
<email>jadahl@gmail.com</email>
</author>
<published>2012-11-03T21:26:11Z</published>
<link rel='alternate' type='text/html' href='http://git.sudomsg.com/mirror/wayland/commit/?id=1801cdc41ac0653e4ff0f0cc4eeaf373ae40edf4'/>
<id>urn:sha1:1801cdc41ac0653e4ff0f0cc4eeaf373ae40edf4</id>
<content type='text'>
Check that after a callback removes a proxy that most likely will have
several events queued up with the same target proxy, no more callbacks
are invoked.

Signed-off-by: Jonas Ådahl &lt;jadahl@gmail.com&gt;
</content>
</entry>
</feed>
