aboutsummaryrefslogtreecommitdiff
path: root/launcher/modplatform/modrinth/ModrinthAPI.h
diff options
context:
space:
mode:
authorflow <thiagodonato300@gmail.com>2022-03-08 11:12:35 -0300
committerflow <thiagodonato300@gmail.com>2022-03-14 17:43:36 -0300
commit8409aa2571d57f015a634a220107d199e88ba2fd (patch)
treee57a9be808be1f4a19b7ae31b1ba846ca53a111b /launcher/modplatform/modrinth/ModrinthAPI.h
parentb3c2a56ece925bc6fe327b8824c50f194610b5b9 (diff)
downloadPrismLauncher-8409aa2571d57f015a634a220107d199e88ba2fd.tar.gz
PrismLauncher-8409aa2571d57f015a634a220107d199e88ba2fd.tar.bz2
PrismLauncher-8409aa2571d57f015a634a220107d199e88ba2fd.zip
tidy: Fix clang-tidy issues on files changed in this PR
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.
Diffstat (limited to 'launcher/modplatform/modrinth/ModrinthAPI.h')
-rw-r--r--launcher/modplatform/modrinth/ModrinthAPI.h10
1 files changed, 5 insertions, 5 deletions
diff --git a/launcher/modplatform/modrinth/ModrinthAPI.h b/launcher/modplatform/modrinth/ModrinthAPI.h
index 1cc51ff2..cf4dec1a 100644
--- a/launcher/modplatform/modrinth/ModrinthAPI.h
+++ b/launcher/modplatform/modrinth/ModrinthAPI.h
@@ -6,10 +6,10 @@
class ModrinthAPI : public NetworkModAPI {
public:
- inline QString getAuthorURL(const QString& name) const { return "https://modrinth.com/user/" + name; };
+ inline auto getAuthorURL(const QString& name) const -> QString { return "https://modrinth.com/user/" + name; };
private:
- inline QString getModSearchURL(SearchArgs& args) const override
+ inline auto getModSearchURL(SearchArgs& args) const -> QString override
{
if (!validateModLoader(args.mod_loader)) {
qWarning() << "Modrinth only have Forge and Fabric-compatible mods!";
@@ -30,12 +30,12 @@ class ModrinthAPI : public NetworkModAPI {
.arg(args.version);
};
- inline QString getVersionsURL(const QString& addonId) const override
+ inline auto getVersionsURL(const QString& addonId) const -> QString override
{
return QString("https://api.modrinth.com/v2/project/%1/version").arg(addonId);
};
- inline QString getModLoaderString(ModLoaderType modLoader) const
+ inline auto getModLoaderString(ModLoaderType modLoader) const -> QString
{
switch (modLoader) {
case Any:
@@ -49,7 +49,7 @@ class ModrinthAPI : public NetworkModAPI {
}
}
- inline bool validateModLoader(ModLoaderType modLoader) const
+ inline auto validateModLoader(ModLoaderType modLoader) const -> bool
{
return modLoader == Any || modLoader == Forge || modLoader == Fabric;
}