GothenburgBitFactory/taskwarrior
1
0
Fork 0
mirror of https://github.com/GothenburgBitFactory/taskwarrior.git synced 2025-09-08 15:20:36 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Revert "[clang-tidy] using dynamic_cast for derived classes"

Browse source 
This reverts commit a02754159c.
This commit is contained in:
Paul Beckingham 2020-12-05 16:18:15 -05:00
parent 6ad1b7d0da
commit 623d5ceb59
2 changed files with 15 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

8
src/Hooks.cpp
View file

@ -432,13 +432,13 @@ void Hooks::assertValidJSON (
throw 0; throw 0;
} }
if ((dynamic_cast<json::object*>(root))->_data.find ("description") == (dynamic_cast<json::object*>(root))->_data.end ()) if (((json::object*)root)->_data.find ("description") == ((json::object*)root)->_data.end ())
{ {
Context::getContext ().error (format (STRING_HOOK_ERROR_NODESC, Path (script).name ())); Context::getContext ().error (format (STRING_HOOK_ERROR_NODESC, Path (script).name ()));
throw 0; throw 0;
} }
if ((dynamic_cast<json::object*>(root))->_data.find ("uuid") == (dynamic_cast<json::object*>(root))->_data.end ()) if (((json::object*)root)->_data.find ("uuid") == ((json::object*)root)->_data.end ())
{ {
Context::getContext ().error (format (STRING_HOOK_ERROR_NOUUID, Path (script).name ())); Context::getContext ().error (format (STRING_HOOK_ERROR_NOUUID, Path (script).name ()));
throw 0; throw 0;
@ -486,7 +486,7 @@ void Hooks::assertSameTask (
for (auto& i : input) for (auto& i : input)
{ {
auto* root_obj = dynamic_cast<json::object*>(json::parse (i)); auto* root_obj = (json::object*)json::parse (i);
// If there is no UUID at all. // If there is no UUID at all.
auto u = root_obj->_data.find ("uuid"); auto u = root_obj->_data.find ("uuid");
@ -497,7 +497,7 @@ void Hooks::assertSameTask (
throw 0; throw 0;
} }
auto* up = dynamic_cast<json::string*>( u->second); auto* up = (json::string*) u->second;
auto text = up->dump (); auto text = up->dump ();
Lexer::dequote (text); Lexer::dequote (text);
std::string json_uuid = json::decode (text); std::string json_uuid = json::decode (text);

22
src/Task.cpp
View file

@ -637,7 +637,7 @@ void Task::parseJSON (const std::string& line)
json::value* root = json::parse (line); json::value* root = json::parse (line);
if (root && if (root &&
root->type () == json::j_object) root->type () == json::j_object)
parseJSON (dynamic_cast<json::object*>( root)); parseJSON ((json::object*) root);
delete root; delete root;
} }
@ -681,10 +681,10 @@ void Task::parseJSON (const json::object* root_obj)
// Tags are an array of JSON strings. // Tags are an array of JSON strings.
else if (i.first == "tags" && i.second->type() == json::j_array) else if (i.first == "tags" && i.second->type() == json::j_array)
{ {
auto* tags = dynamic_cast<json::array*>(i.second); auto* tags = (json::array*)i.second;
for (auto& t : tags->_data) for (auto& t : tags->_data)
{ {
auto* tag = dynamic_cast<json::string*>(t); auto* tag = (json::string*)t;
addTag (tag->_data); addTag (tag->_data);
} }
} }
@ -695,7 +695,7 @@ void Task::parseJSON (const json::object* root_obj)
// removed in a later release. // removed in a later release.
else if (i.first == "tags" && i.second->type() == json::j_string) else if (i.first == "tags" && i.second->type() == json::j_string)
{ {
auto* tag = dynamic_cast<json::string*>(i.second); auto* tag = (json::string*)i.second;
addTag (tag->_data); addTag (tag->_data);
} }
@ -704,10 +704,10 @@ void Task::parseJSON (const json::object* root_obj)
// See other 2016-02-21 comments for details. // See other 2016-02-21 comments for details.
else if (i.first == "depends" && i.second->type() == json::j_array) else if (i.first == "depends" && i.second->type() == json::j_array)
{ {
auto* deps = dynamic_cast<json::array*>(i.second); auto* deps = (json::array*)i.second;
for (auto& t : deps->_data) for (auto& t : deps->_data)
{ {
auto* dep = dynamic_cast<json::string*>(t); auto* dep = (json::string*)t;
addDependency (dep->_data); addDependency (dep->_data);
} }
} }
@ -716,7 +716,7 @@ void Task::parseJSON (const json::object* root_obj)
// 2016-02-21: Deprecated - see other 2016-02-21 comments for details. // 2016-02-21: Deprecated - see other 2016-02-21 comments for details.
else if (i.first == "depends" && i.second->type() == json::j_string) else if (i.first == "depends" && i.second->type() == json::j_string)
{ {
auto* deps = dynamic_cast<json::string*>(i.second); auto* deps = (json::string*)i.second;
auto uuids = split (deps->_data, ','); auto uuids = split (deps->_data, ',');
for (const auto& uuid : uuids) for (const auto& uuid : uuids)
@ -749,12 +749,12 @@ void Task::parseJSON (const json::object* root_obj)
{ {
std::map <std::string, std::string> annos; std::map <std::string, std::string> annos;
auto* atts = dynamic_cast<json::array*>(i.second); auto* atts = (json::array*)i.second;
for (auto& annotations : atts->_data) for (auto& annotations : atts->_data)
{ {
auto* annotation = dynamic_cast<json::object*>(annotations); auto* annotation = (json::object*)annotations;
json::string* when = dynamic_cast<json::string*>(annotation->_data["entry"]); json::string* when = (json::string*)annotation->_data["entry"];
json::string* what = dynamic_cast<json::string*>(annotation->_data["description"]); json::string* what = (json::string*)annotation->_data["description"];
if (! when) if (! when)
throw format ("Annotation is missing an entry date: {1}", root_obj-> dump ()); throw format ("Annotation is missing an entry date: {1}", root_obj-> dump ());