pre-commit

Author: Lucas-Haubert

examples/python/osqp_solve_dense_qp.py

@@ -1,6 +1,5 @@
 import proxsuite
 import numpy as np
-import scipy.sparse as spa
 from util import generate_mixed_qp
 
 

include/proxsuite/helpers/tl-optional.hpp

@@ -169,7 +169,7 @@ template<class T, class Ret, class... Args>
 struct is_pointer_to_non_const_member_func<Ret (T::*)(Args...)> : std::true_type
 {};
 template<class T, class Ret, class... Args>
-struct is_pointer_to_non_const_member_func<Ret (T::*)(Args...) &>
+struct is_pointer_to_non_const_member_func<Ret (T::*)(Args...)&>
   : std::true_type
 {};
 template<class T, class Ret, class... Args>

include/proxsuite/linalg/veg/internal/macros.hpp

@@ -278,9 +278,9 @@
     using NothrowTestExpr = ::proxsuite::linalg::veg::meta::false_type;        \
   };                                                                           \
   template<__VEG_PP_REMOVE_PAREN1(Tpl)>                                        \
-  struct test_sfinae_##                                                        \
-    Name<::proxsuite::linalg::veg::meta::void_t<decltype((Expr))>,             \
-         __VEG_PP_REMOVE_PAREN1(TplNames)>                                     \
+  struct test_sfinae_##Name<                                                   \
+    ::proxsuite::linalg::veg::meta::void_t<decltype((Expr))>,                  \
+    __VEG_PP_REMOVE_PAREN1(TplNames)>                                          \
   {                                                                            \
     using ExprType = decltype((Expr));                                         \
     using TestExpr =                                                           \
@@ -1249,7 +1249,7 @@ extract_chars_expr(LiteralType /*unused*/) -> typename ExtractCharsImplExpr<
   (sizeof(__VEG_PP_CAT(u8, __VEG_PP_STRINGIZE(MemberPtr))) - 1),
 
 #define __VEG_IMPL_STRUCT_SETUP(PClass, ...) /* NOLINT */                      \
-  void _veg_lib_name_test() && noexcept                                        \
+  void _veg_lib_name_test()&& noexcept                                         \
   {                                                                            \
     static_assert(                                                             \
       VEG_CONCEPT(same<decltype(this), __VEG_PP_REMOVE_PAREN(PClass)*>),       \

include/proxsuite/linalg/veg/internal/prologue.hpp

@@ -55,15 +55,15 @@
   ~Class() = default;                                                          \
   Class(Class&&) = default;                                                    \
   explicit Class(Class const&) = default;                                      \
-  auto operator=(Class&&) &->Class& = default;                                 \
-  auto operator=(Class const&) &->Class& = default
+  auto operator=(Class&&)&->Class& = default;                                  \
+  auto operator=(Class const&)&->Class& = default
 
 #define VEG_NO_COPY(Class)                                                     \
   ~Class() = default;                                                          \
   Class(Class&&) = default;                                                    \
   Class(Class const&) = delete;                                                \
-  auto operator=(Class&&) &->Class& = default;                                 \
-  auto operator=(Class const&) &->Class& = delete
+  auto operator=(Class&&)&->Class& = default;                                  \
+  auto operator=(Class const&)&->Class& = delete
 
 #ifdef VEG_WITH_CXX14_SUPPORT
 #define VEG_CPP14(...) __VA_ARGS__

test/doctest/doctest.hpp

@@ -1563,8 +1563,7 @@ namespace detail {
 #endif // DOCTEST_CONFIG_TREAT_CHAR_STAR_AS_STRING
 // clang-format on
 
-struct DOCTEST_INTERFACE TestFailureException
-{};
+struct DOCTEST_INTERFACE TestFailureException{};
 
 DOCTEST_INTERFACE bool
 checkIfShouldThrow(assertType::Enum at);
@@ -2670,8 +2669,9 @@ registerReporter(const char* name, int priority, bool isReporter)
 
 // for subcases
 #define DOCTEST_SUBCASE(name)                                                  \
-  if (const doctest::detail::Subcase& DOCTEST_ANONYMOUS(DOCTEST_ANON_SUBCASE_) \
-        DOCTEST_UNUSED = doctest::detail::Subcase(name, __FILE__, __LINE__))
+  if (const doctest::detail::Subcase &                                         \
+        DOCTEST_ANONYMOUS(DOCTEST_ANON_SUBCASE_)                               \
+          DOCTEST_UNUSED = doctest::detail::Subcase(name, __FILE__, __LINE__))
 
 // for grouping tests in test suites by using code blocks
 #define DOCTEST_TEST_SUITE_IMPL(decorators, ns_name)                           \
@@ -5446,8 +5446,7 @@ DOCTEST_CLANG_SUPPRESS_WARNING_POP
 DOCTEST_GCC_SUPPRESS_WARNING_POP
 DOCTEST_MSVC_SUPPRESS_WARNING_POP
 
-Subcase::
-operator bool() const
+Subcase::operator bool() const
 {
   return m_entered;
 }

test/src/osqp_dense_qp_solve.py

@@ -348,8 +348,8 @@ def test_initializing_with_None(self):
         A = None
         b = None
         C = None
-        u = None
-        l = None
+        _u = None
+        _l = None
 
         results = proxsuite.osqp.dense.solve(
             H,

test/src/osqp_dense_qp_wrapper.py

@@ -1,10 +1,6 @@
 #
 # Copyright (c) 2022-2023, INRIA
 #
-import proxsuite
-import numpy as np
-import scipy.sparse as spa
-import unittest
 
 # TODO test:
 # Huge amount of tests -> do it once the previous ones are ok