mirror of
https://github.com/Chatterino/chatterino2.git
synced 2024-11-13 19:49:51 +01:00
fdb0a1582c
* Remove unused include util/Helpers.hpp * SplitContainer::setTag fix parameter naming * autofy/constify where possible * More const auto ptr magicifying * Make SplitNode::Type an enum class * Move QuickSwitcherPopup includes from header to source file * Remove unused DropRegion code * use empty() instead of size() == 0 * Add curly braces everywhere * Remove useless reinterpret_cast It was casting Node* to Node* * Clarify that the connect is QObject::connect * SplitContainer::setSelected fix parameter naming * Rename function variables to remove unneccesary underscore Also move addSpacing parameter out of the layout function * emplace_back where possible * Name parameters * Remove ineffective const from return type * Make node getters const * Flatten Node::releaseSplit * Rename in-function variable to match code style * [ACTUAL CODE CHANGE/MOVE] Move clamp logic to its own function * name params * applyFromDescriptorRecursively: rename node param to baseNode * [ACTUAL CODE CHANGE/MOVE] Remove the many overloads for append/insertSplit This utilizes the C++20 designed initializers aggregate initialization feature * Remove unused includes * [ACTUAL CODE CHANGE] Clean up dragging logic There's no need to keep a pointer around to which split is being dragged, it's already stored in the QDropEvent source() * UNRELATED .clang-tidy: Only suggest UPPER_CASE for constant global variables * Remove unused SplitContainer::getSplitCount function * Use std::max in Node's clamp function * Remove test code * DraggedSplit.hpp: remove unused include * Split `setDraggingSplit` into two functions, `startDraggingSplit` and `stopDraggingSplit`
57 lines
2.1 KiB
YAML
57 lines
2.1 KiB
YAML
Checks: '-*,
|
|
clang-diagnostic-*,
|
|
llvm-*,
|
|
misc-*,
|
|
-misc-unused-parameters,
|
|
readability-identifier-naming,
|
|
-llvm-header-guard,
|
|
-llvm-include-order,
|
|
modernize-*,
|
|
readability-*,
|
|
performance-*,
|
|
misc-*,
|
|
bugprone-*,
|
|
cert-*,
|
|
cppcoreguidelines-*,
|
|
-cppcoreguidelines-pro-type-cstyle-cast,
|
|
-cppcoreguidelines-pro-bounds-pointer-arithmetic,
|
|
-cppcoreguidelines-pro-bounds-array-to-pointer-decay,
|
|
-cppcoreguidelines-pro-type-member-init,
|
|
-cppcoreguidelines-owning-memory,
|
|
-cppcoreguidelines-avoid-magic-numbers,
|
|
-readability-magic-numbers,
|
|
-performance-noexcept-move-constructor,
|
|
-misc-non-private-member-variables-in-classes,
|
|
-cppcoreguidelines-non-private-member-variables-in-classes,
|
|
-cppcoreguidelines-special-member-functions,
|
|
-modernize-use-nodiscard,
|
|
-modernize-use-trailing-return-type,
|
|
-readability-identifier-length,
|
|
-readability-function-cognitive-complexity,
|
|
-bugprone-easily-swappable-parameters,
|
|
-cert-err58-cpp,
|
|
'
|
|
CheckOptions:
|
|
- key: readability-identifier-naming.ClassCase
|
|
value: CamelCase
|
|
- key: readability-identifier-naming.EnumCase
|
|
value: CamelCase
|
|
- key: readability-identifier-naming.FunctionCase
|
|
value: camelBack
|
|
- key: readability-identifier-naming.MemberCase
|
|
value: camelBack
|
|
- key: readability-identifier-naming.PrivateMemberIgnoredRegexp
|
|
value: .*
|
|
- key: readability-identifier-naming.PrivateMemberSuffix
|
|
value: _
|
|
- key: readability-identifier-naming.ProtectedMemberSuffix
|
|
value: _
|
|
- key: readability-identifier-naming.UnionCase
|
|
value: CamelCase
|
|
- key: readability-identifier-naming.GlobalConstantCase
|
|
value: UPPER_CASE
|
|
- key: readability-identifier-naming.VariableCase
|
|
value: camelBack
|
|
- key: readability-implicit-bool-conversion.AllowPointerConditions
|
|
value: true
|