Change FIXMEs into TODOs where appropriate

This commit is contained in:
Alex Beregszaszi 2019-01-29 00:35:23 +00:00
parent fa2bb01da3
commit ee09d54525
3 changed files with 5 additions and 5 deletions

View File

@ -3027,8 +3027,8 @@ u256 FunctionType::externalIdentifier() const
bool FunctionType::isPure() const bool FunctionType::isPure() const
{ {
// FIXME: replace this with m_stateMutability == StateMutability::Pure once // TODO: replace this with m_stateMutability == StateMutability::Pure once
// the callgraph analyzer is in place // the callgraph analyzer is in place
return return
m_kind == Kind::KECCAK256 || m_kind == Kind::KECCAK256 ||
m_kind == Kind::ECRecover || m_kind == Kind::ECRecover ||

View File

@ -473,7 +473,7 @@ eth::LinkerObject const& CompilerStack::runtimeObject(string const& _contractNam
return contract(_contractName).runtimeObject; return contract(_contractName).runtimeObject;
} }
/// FIXME: cache this string /// TODO: cache this string
string CompilerStack::assemblyString(string const& _contractName, StringMap _sourceCodes) const string CompilerStack::assemblyString(string const& _contractName, StringMap _sourceCodes) const
{ {
if (m_stackState != CompilationSuccessful) if (m_stackState != CompilationSuccessful)
@ -486,7 +486,7 @@ string CompilerStack::assemblyString(string const& _contractName, StringMap _sou
return string(); return string();
} }
/// FIXME: cache the JSON /// TODO: cache the JSON
Json::Value CompilerStack::assemblyJSON(string const& _contractName, StringMap _sourceCodes) const Json::Value CompilerStack::assemblyJSON(string const& _contractName, StringMap _sourceCodes) const
{ {
if (m_stackState != CompilationSuccessful) if (m_stackState != CompilationSuccessful)

View File

@ -50,7 +50,7 @@ void InlinableExpressionFunctionFinder::operator()(FunctionDefinition const& _fu
Assignment const& assignment = boost::get<Assignment>(bodyStatement); Assignment const& assignment = boost::get<Assignment>(bodyStatement);
if (assignment.variableNames.size() == 1 && assignment.variableNames.front().name == retVariable) if (assignment.variableNames.size() == 1 && assignment.variableNames.front().name == retVariable)
{ {
// FIXME: use code size metric here // TODO: use code size metric here
// We cannot overwrite previous settings, because this function definition // We cannot overwrite previous settings, because this function definition
// would not be valid here if we were searching inside a functionally inlinable // would not be valid here if we were searching inside a functionally inlinable