| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
| |
Fixes the following build error under musl libc:
../../../tests/utils/qmain.cpp: In function 'int main(int, char**)':
../../../tests/utils/qmain.cpp:47:5: error: 'bindtextdomain' was not declared in this scope
47 | bindtextdomain(GETTEXT_PACKAGE, LOCALEDIR);
| ^~~~~~~~~~~~~~
../../../tests/utils/qmain.cpp:48:5: error: 'textdomain' was not declared in this scope
48 | textdomain(GETTEXT_PACKAGE);
| ^~~~~~~~~~
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
CMake's documentation says [1]:
> Do not list the output in more than one independent target that may
> build in parallel or the two instances of the rule may conflict
> (instead use the add_custom_target() command to drive the command and
> make the other targets depend on that one).
Because gschemas.compiled will be referenced multiple times
independently, a level of redirection is needed to prevent CMake from
generating the rule for the file multiple times.
This is encountered in ayatana-indicator-power as well, and fixed in a
similar way [2].
[1] https://cmake.org/cmake/help/latest/command/add_custom_command.html#generating-files
[2] https://github.com/AyatanaIndicators/ayatana-indicator-power/commit/9a90f932e51db486b166dd38f00e4186e40a1aee
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
unit tests.
Additionally, internationalize the org.ayatana.display GSettings schema.
Fixes https://github.com/AyatanaIndicators/ayatana-indicator-display/issues/39
|
| |
|
| |
|
| |
|
|
|
|
| |
- Making the problematic parts "const" produce even more warnings, so I'm considering this a false positive and disabling it
|
|
|
|
| |
Mark unused adbd_server and usb_manager as such.
|
|
|
|
|
|
|
|
| |
- src/rotation-lock.cpp: Fix double declaration of create_desktop_menu
- tests/CMakeLists.txt: Add uninitDerivedMemberVar and unmatchedSuppression suppressions to cppcheck
- tests/unit/rotation-lock-test.cpp: Fix and expand test
fixes https://github.com/AyatanaIndicators/ayatana-indicator-display/issues/24
|
| |
|
|
|
|
| |
--suppress parameters.
|
|
|
|
| |
is used.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
explicitly declared constructor.
Resolves issues like these:
In file included from /rootdir/tests/unit/rotation-lock-test.cpp:20:
In file included from /rootdir/tests/utils/test-dbus-fixture.h:22:
In file included from /rootdir/tests/utils/glib-fixture.h:30:
In file included from /usr/src/googletest/googletest/include/gtest/gtest.h:62:
/usr/src/googletest/googletest/include/gtest/internal/gtest-internal.h:472:44: error: call to implicitly-deleted default constructor of 'RotationLockFixture_CheckIndicator_Test'
Test* CreateTest() override { return new TestClass; }
^
/rootdir/tests/unit/rotation-lock-test.cpp:46:1: note: in instantiation of member function 'testing::internal::TestFactoryImpl<RotationLockFixture_CheckIndicator_Test>::CreateTest' requested here
TEST_F(RotationLockFixture, CheckIndicator)
^
/usr/src/googletest/googletest/include/gtest/gtest.h:2381:3: note: expanded from macro 'TEST_F'
GTEST_TEST_(test_fixture, test_name, test_fixture, \
^
/usr/src/googletest/googletest/include/gtest/internal/gtest-internal.h:1565:15: note: expanded from macro 'GTEST_TEST_'
new ::testing::internal::TestFactoryImpl<GTEST_TEST_CLASS_NAME_( \
^
/rootdir/tests/unit/rotation-lock-test.cpp:46:1: note: explicitly defaulted function was implicitly deleted here
/usr/src/googletest/googletest/include/gtest/gtest.h:2381:3: note: expanded from macro 'TEST_F'
GTEST_TEST_(test_fixture, test_name, test_fixture, \
^
/usr/src/googletest/googletest/include/gtest/internal/gtest-internal.h:1544:5: note: expanded from macro 'GTEST_TEST_'
GTEST_TEST_CLASS_NAME_(test_suite_name, test_name)() = default; \
^
/usr/src/googletest/googletest/include/gtest/internal/gtest-internal.h:1533:3: note: expanded from macro 'GTEST_TEST_CLASS_NAME_'
test_suite_name##_##test_name##_Test
^
<scratch space>:198:1: note: expanded from here
RotationLockFixture_CheckIndicator_Test
^
/rootdir/tests/unit/rotation-lock-test.cpp:46:1: note: default constructor of 'RotationLockFixture_CheckIndicator_Test' is implicitly deleted because base class 'RotationLockFixture' has a deleted default constructor
/usr/src/googletest/googletest/include/gtest/gtest.h:2381:3: note: expanded from macro 'TEST_F'
GTEST_TEST_(test_fixture, test_name, test_fixture, \
^
/usr/src/googletest/googletest/include/gtest/internal/gtest-internal.h:1542:9: note: expanded from macro 'GTEST_TEST_'
: public parent_class { \
^
/rootdir/tests/unit/rotation-lock-test.cpp:24:28: note: default constructor of 'RotationLockFixture' is implicitly deleted because base class 'TestDBusFixture' has a deleted default constructor
class RotationLockFixture: public TestDBusFixture
^
/rootdir/tests/utils/test-dbus-fixture.h:32:5: note: explicitly defaulted function was implicitly deleted here
TestDBusFixture() =default;
^
/rootdir/tests/utils/test-dbus-fixture.h:69:36: note: default constructor of 'TestDBusFixture' is implicitly deleted because field 'service_dirs' of const-qualified type 'const std::vector<std::string>' (aka 'const vector<basic_string<char>>') would not be initialized
const std::vector<std::string> service_dirs;
^
2 warnings and 1 error generated.
|
|
|
|
| |
--suppress=unusedFunction. Be less strict and make the cppcheck pass.
|
| |
|
|
|
|
| |
(which is one of our unit tests).
|
| |
|
| |
|
| |
|
| |
|
|\ |
|
| |
| |
| |
| | |
each time there's a usb disconnect
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| | |
creating the adb socket in AdbdServer's ctor instead of in its worker thread.
|
| |
| |
| |
| | |
from an 'is_active' bool to a three-value state of active, inactive, and unavailable
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| | |
test
|
|/ |
|
| |
|
|
|
|
| |
prompts
|
| |
|
| |
|
|
|
|
| |
on subsequent requests
|
| |
|
|
|
|
| |
give the test file and line number
|
|
|
|
| |
the prompt.
|
|
|
|
| |
fixtures' helper/util code
|
| |
|
|
|
|
| |
in one place
|