aboutsummaryrefslogtreecommitdiffstats
path: root/libsolidity
diff options
context:
space:
mode:
authorchriseth <chris@ethereum.org>2018-07-13 00:00:05 +0800
committerGitHub <noreply@github.com>2018-07-13 00:00:05 +0800
commit81271801b4560c15911b69a9f6d7e0530359f523 (patch)
tree9e52f36778ee877b1dc2fdf9b532b8aae39f369c /libsolidity
parent0dd79bc1724bfd5031dd6d1bc3b4c1eda283b5f3 (diff)
parentdf8bbffd8e5b16d29f71860e1c3c851dd65693c4 (diff)
downloaddexon-solidity-81271801b4560c15911b69a9f6d7e0530359f523.tar.gz
dexon-solidity-81271801b4560c15911b69a9f6d7e0530359f523.tar.zst
dexon-solidity-81271801b4560c15911b69a9f6d7e0530359f523.zip
Merge pull request #4478 from ethereum/requireStorageLocation
Turn missing storage locations into an error.
Diffstat (limited to 'libsolidity')
-rw-r--r--libsolidity/analysis/ReferencesResolver.cpp17
1 files changed, 4 insertions, 13 deletions
diff --git a/libsolidity/analysis/ReferencesResolver.cpp b/libsolidity/analysis/ReferencesResolver.cpp
index 58b659f7..dfcbf888 100644
--- a/libsolidity/analysis/ReferencesResolver.cpp
+++ b/libsolidity/analysis/ReferencesResolver.cpp
@@ -377,19 +377,10 @@ void ReferencesResolver::endVisit(VariableDeclaration const& _variable)
{
typeLoc = DataLocation::Storage;
if (_variable.isLocalVariable())
- {
- if (_variable.sourceUnit().annotation().experimentalFeatures.count(ExperimentalFeature::V050))
- typeError(
- _variable.location(),
- "Data location must be specified as either \"memory\" or \"storage\"."
- );
- else
- m_errorReporter.warning(
- _variable.location(),
- "Variable is declared as a storage pointer. "
- "Use an explicit \"storage\" keyword to silence this warning."
- );
- }
+ typeError(
+ _variable.location(),
+ "Data location must be specified as either \"memory\" or \"storage\"."
+ );
}
}
else