Merge lp:~andreas-pokorny/mir/libinput-platform into lp:mir
| Status: | Merged |
|---|---|
| Approved by: | Andreas Pokorny on 2015-09-22 |
| Approved revision: | 2420 |
| Merged at revision: | 2956 |
| Proposed branch: | lp:~andreas-pokorny/mir/libinput-platform |
| Merge into: | lp:mir |
| Prerequisite: | lp:~andreas-pokorny/mir/remove-dispatchable-from-input-device |
| Diff against target: |
2503 lines (+2279/-9) 25 files modified
CMakeLists.txt (+1/-0) debian/control (+1/-0) include/test/mir/test/event_matchers.h (+45/-0) src/CMakeLists.txt (+1/-0) src/platforms/CMakeLists.txt (+2/-0) src/platforms/evdev/CMakeLists.txt (+29/-0) src/platforms/evdev/input_modifier_utils.cpp (+8/-5) src/platforms/evdev/input_modifier_utils.h (+1/-1) src/platforms/evdev/libinput_device.cpp (+347/-0) src/platforms/evdev/libinput_device.h (+90/-0) src/platforms/evdev/libinput_device_ptr.cpp (+30/-0) src/platforms/evdev/libinput_device_ptr.h (+40/-0) src/platforms/evdev/libinput_ptr.cpp (+47/-0) src/platforms/evdev/libinput_ptr.h (+42/-0) src/platforms/evdev/platform.cpp (+238/-0) src/platforms/evdev/platform.h (+94/-0) src/platforms/evdev/platform_factory.cpp (+68/-0) tests/include/mir/test/doubles/mock_libinput.h (+110/-0) tests/mir_test_doubles/CMakeLists.txt (+1/-0) tests/mir_test_doubles/mock_libinput.cpp (+316/-0) tests/mir_test_framework/CMakeLists.txt (+1/-1) tests/mir_test_framework/fake_input_device_impl.cpp (+2/-2) tests/unit-tests/input/evdev/CMakeLists.txt (+3/-0) tests/unit-tests/input/evdev/test_evdev_input_platform.cpp (+264/-0) tests/unit-tests/input/evdev/test_libinput_device.cpp (+498/-0) |
| To merge this branch: | bzr merge lp:~andreas-pokorny/mir/libinput-platform |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| PS Jenkins bot | continuous-integration | 2015-09-07 | Approve on 2015-09-22 |
| Alexandros Frantzis (community) | Approve on 2015-09-22 | ||
| Kevin DuBois (community) | Approve on 2015-09-17 | ||
| Alan Griffiths | 2015-09-07 | Approve on 2015-09-17 | |
| Cemil Azizoglu (community) | Needs Fixing on 2015-09-11 | ||
|
Review via email:
|
|||
This proposal supersedes a proposal from 2015-08-28.
Commit Message
Add input-evdev.so based on libinput
This adds the evdev platform that shall replace InputReader, EventHub and the InputMappers. The platform is not yet selected by default. Code to probe and select the right platform is still missing for the input side.
Description of the Change
This adds the libinput platform as input-evdev.so
It does not yet use the extended libinput_
You can test that in the build dir with --platform-
| Andreas Pokorny (andreas-pokorny) wrote : | # |
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2402
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2403
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Andreas Pokorny (andreas-pokorny) wrote : | # |
This MP needs a newer libinput in vivid+overlay. Silo for that is on the way.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2404
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2406
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2406
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2406
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2406
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2406
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2404
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Alan Griffiths (alan-griffiths) wrote : | # |
Text conflict in include/
1 conflicts encountered
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2408
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2407
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 2409. By Andreas Pokorny on 2015-09-07
-
update to newer libinput version
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2409
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 2410. By Andreas Pokorny on 2015-09-09
-
restore unit tests that got lost during the last branch merging spree
- 2411. By Andreas Pokorny on 2015-09-10
-
Resurrect touch tests and fix none-state of input event modifier
- 2412. By Andreas Pokorny on 2015-09-10
-
header not needed
- 2413. By Andreas Pokorny on 2015-09-10
-
merge prereq branch
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2412
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2413
http://
Executed test runs:
SUCCESS: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2413
http://
Executed test runs:
SUCCESS: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Cemil Azizoglu (cemil-azizoglu) wrote : | # |
(Haven't completed the review yet.)
1211 +extern "C" mir::UniqueModu
1222 +extern "C" void add_input_
1228 +extern "C" mi::PlatformPri
1238 +extern "C" mir::ModuleProp
Please remove these "extern" keywords in the definition of functions as they cause problems with
abi-compliance-
-------
- 2414. By Andreas Pokorny on 2015-09-14
-
remove extern declaration from function bodies
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2414
http://
Executed test runs:
SUCCESS: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2414
http://
Executed test runs:
SUCCESS: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2414
http://
Executed test runs:
SUCCESS: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Francis Ginther (fginther) wrote : | # |
Andreas, the most recent CI run includes the addition of the overlay PPA for all builds. Please let me know if you have any questions.
- 2415. By Andreas Pokorny on 2015-09-15
-
fix missing exception spec
- 2416. By Andreas Pokorny on 2015-09-15
-
reverting a bunch of unnecessary changes
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:2416
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Alan Griffiths (alan-griffiths) wrote : | # |
A mild "needs fixing"
+ EXPECT_
+ EXPECT_
+ EXPECT_
These magic numbers are hard to interpret. Couldn't we have, for example:
EXPECT_
~~~~
-MirInputEventM
+MirInputEventM
Now the function name disagrees with the return type.
~~~~
+ auto action = ...
Here and elsewhere could be "auto const"
- 2417. By Andreas Pokorny on 2015-09-17
-
review findings fixed
- 2418. By Andreas Pokorny on 2015-09-17
-
merge lp:mir
| Andreas Pokorny (andreas-pokorny) wrote : | # |
> A mild "needs fixing"
>
> + EXPECT_
> handle_
> + EXPECT_
> + EXPECT_
>
> These magic numbers are hard to interpret. Couldn't we have, for example:
>
> EXPECT_
> mir_pointer_
>
Only did that for the new matcher. The other matchers are used frequently. I think we can do the typing of event matchers in separate mps.
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:2418
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Kevin DuBois (kdub) wrote : | # |
LGTM... I have a slight preference for aligning indentation at 4 spaces for multiline instead of open-parenthesis alignment, but thats no reason to block.
| Alexandros Frantzis (afrantzis) wrote : | # |
+ MATCHER_
+ ...
+ if (mir_pointer_
+ return false;
Shouldn't this be != mir_pointer_
+void mie::Platform:
+{
+ log_info("device added %s", dev.devnode());
Wrong log message.
+ mir::log_
+ report-
If the logging report is selected we will get the message twice. The mix of logging and reporting here doesn't seem correct.
+ for(auto ev = next_event(); ev; ev = next_event())
Simpler as 'while (auto ev = next_event()) ...'
- 2419. By Andreas Pokorny on 2015-09-18
-
fixed event matcher (and actually made use of it)
cleanup of log messages
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2419
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 2420. By Andreas Pokorny on 2015-09-18
-
resolve conflict with lp:mir changes
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:2420
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://

ok i just saw two more things I can split out..