8409aa2571
The checks used are roughly the same as the ones proposed in the clang-tidy PR (except perhaps that I used modernize-* instead of listing them individually,though I don't think this caused any readability detriments). In ModrinthModel.cpp and FlameModModel.cpp I ignored the modernize-avoid-c-arrays one, mostly because making the sorts array an std::array would most likely increase the code complexity because of the virtual function. Aside from that, the static_cast warning from Application.h was not dealt with, since it's not in this PR's scope.
26 lines
696 B
C++
26 lines
696 B
C++
#pragma once
|
|
|
|
#include "FlameModPage.h"
|
|
|
|
namespace FlameMod {
|
|
|
|
class ListModel : public ModPlatform::ListModel {
|
|
Q_OBJECT
|
|
|
|
public:
|
|
ListModel(FlameModPage* parent) : ModPlatform::ListModel(parent) {}
|
|
~ListModel() override = default;
|
|
|
|
private:
|
|
void loadIndexedPack(ModPlatform::IndexedPack& m, QJsonObject& obj) override;
|
|
void loadIndexedPackVersions(ModPlatform::IndexedPack& m, QJsonArray& arr) override;
|
|
|
|
auto documentToArray(QJsonDocument& obj) const -> QJsonArray override;
|
|
|
|
// NOLINTNEXTLINE(modernize-avoid-c-arrays)
|
|
static const char* sorts[6];
|
|
inline auto getSorts() const -> const char** override { return sorts; };
|
|
};
|
|
|
|
} // namespace FlameMod
|