Age | Commit message (Collapse) | Author | Files | Lines |
|
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Add a binding to get the current value of an EV_ABS value.
https://bugs.freedesktop.org/show_bug.cgi?id=102615
Signed-off-by: Benjamin Berg <bberg@redhat.com>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
check_evemu_read() is only use locally in this module. It should be static.
Signed-off-by: Martin Kepplinger <martink@posteo.de>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
In case the output file could not have been opened, it is accessed anyways
although it's zero, resulting in a segmentation fault.
So let's free it in the out-path only if allocated before.
Signed-off-by: Martin Kepplinger <martink@posteo.de>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
We had incremented the argv pointer once too often here. What was read
is some buffer outside of our scope, not the next argument after the
device. Right before, "prefix" is already set up to point to the user's
output file path. Let's use it.
I guess people work around and always use redirection of stdout in order
to get a file instead. This makes writing directly work.
Signed-off-by: Martin Kepplinger <martink@posteo.de>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
In case the output file cannot be opened, the message is the only thing
that gets printed and really should contain a newline in order to have
a sane commandline prompt, for example.
Signed-off-by: Martin Kepplinger <martink@posteo.de>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
It's supposed to say "current" instead of "curent".
Signed-off-by: Martin Kepplinger <martink@posteo.de>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
output and autorestart are only used in evemu-record. Declaring them static
makes things a little more readable.
Signed-off-by: Martin Kepplinger <martink@posteo.de>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Related to https://bugs.freedesktop.org/show_bug.cgi?id=101150
For a negatice time delta (can happen on autoresume, see the bug) we end up
printing +-25ms. This patch doesn't fix the negative time delta but at least
switches it to just print -25ms.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
It's a false positive because we only ever print /dev/input/eventXX but let's
shut it up anyway.
find_event_devices.c: In function ‘find_event_devices’:
find_event_devices.c:65:9: warning: ‘%s’ directive output may be truncated writing up to 255 bytes into a region of size 53 [-Wformat-truncation=]
"%s/%s", DEV_INPUT_EVENT, namelist[i]->d_name);
^~
find_event_devices.c:64:3: note: ‘snprintf’ output between 12 and 267 bytes into a destination of size 64
snprintf(fname, sizeof(fname),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
"%s/%s", DEV_INPUT_EVENT, namelist[i]->d_name);
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Python3 uses unicode and byte arrays need to be explicitly encoded/decoded in
Python3 to avoid the ctypes TypeError when converting between python strings
and C strings.
https://bugs.freedesktop.org/show_bug.cgi?id=97458
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
NULL-pointers are None, not 0. This only worked because we kept falling
through until some other later condition triggered and happened to return the
correct value at all times (usually because the type/code ended up as -1)
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
A fairly subtle difference, but "is None" is preferred.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Allows running through the event sequence more than once without any extra
handling. In modern recordings the description and the event file is in the
same file and there is no extra fd handling for the events. So simple code to
check events would look like this:
d = evemu.Device("/path/to/file", create=False)
for e in d.events():
check_for_something()
for e in d.events():
check_for_something_else()
Simply rewinding the fd is sufficient here to avoid the caller having to keep
a copy of the events.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Martin <consume.noise@gmail.com>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Martin <consume.noise@gmail.com>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
https://bugs.freedesktop.org/show_bug.cgi?id=97956
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Martin <consume.noise@gmail.com>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Martin <consume.noise@gmail.com>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Needed often enough, might as well print it
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Martin <consume.noise@gmail.com>
|
|
It's already a fatal error, print it that way.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Martin <consume.noise@gmail.com>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Martin <consume.noise@gmail.com>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Martin <consume.noise@gmail.com>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Martin <consume.noise@gmail.com>
|
|
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
$ evemu-play touchpad.evemu
will create the device from the recording and prompt the user to hit enter
before replaying the sequence. Re-playing the same sequence multiple times
just requires hitting enter again.
https://bugs.freedesktop.org/show_bug.cgi?id=96688
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
No real code changes, this just merges the two files in preparation for a
multi-mode evemu-play.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
bugzilla attachments sometimes have CRLF as end of line, make sure we ignore
that for the device name. Otherwise our device name contains the \r, affecting
printfs.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Keep any file that has events in it and keep the last file (i.e. when
evemu-record was cancelled).
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Uses the existing evemu timeout feature where evemu_record() quits after a
given timeout. A call of
evemu-record --autorestart 10 /dev/input/event4 scroll.evemu
will use the filename as prefix and create a timestamped output file (e.g.
scroll.evemu.2016-02-25-09:13). Until the 10s inactivity timeout is hit, all
events are recorded to that file. After that, the file is closed with a note
at the bottom and a new file is started.
This enables a user to leave evemu running in the background for a prolonged
time and recover the most recent recording after triggering a bug without
having to wade through three days of recordings.
https://bugs.freedesktop.org/show_bug.cgi?id=93752
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
This doesn't make sense just yet, it's prep work for an upcoming patch to call
describe_device multiple times in the evemu-record case.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Make sure the output file has at least the description, even if we didn't get
any events.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
argv[2] specifies the output file to record to (stdout by default).
evemu-record /dev/input/event0 somefile
This recording was missing the device description tough.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
The v2.3.0 was missing the doc files. Bumping the release.
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
shush, coverity, shush
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
hush little coverity, don't you cry,
that scanf error is not gonna squeeze by
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Coverity complains. There isn't really a case where this should go wrong, so
we can silently ignore the error case.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
To shut up coverity
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Introduced in 79b29f097e97
And indent correctly while we're here.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|
|
Makes the cleanup path nicer and stops coverity warnings. The tools all call
evemu_destroy() unconditionally, even when evemu_create() failed.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
|