cerc-io/solidity
16
0
Fork 0
mirror of https://github.com/ethereum/solidity synced 2023-10-03 13:03:40 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Remove non-0.5.0 warning for emit keyword (make it mandatory)

Browse Source
This commit is contained in:
Alex Beregszaszi 2018-06-11 18:10:24 +01:00 committed by chriseth
parent 503eb8caa5
commit 80b7d36187
7 changed files with 7 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
libsolidity/analysis/TypeChecker.cpp
View File

@ -1712,12 +1712,7 @@ bool TypeChecker::visit(FunctionCall const& _functionCall)
m_errorReporter.typeError(_functionCall.location(), "\"suicide\" has been deprecated in favour of \"selfdestruct\"");
}
if (!m_insideEmitStatement && functionType->kind() == FunctionType::Kind::Event)
{
if (m_scope->sourceUnit().annotation().experimentalFeatures.count(ExperimentalFeature::V050))
m_errorReporter.typeError(_functionCall.location(), "Event invocations have to be prefixed by \"emit\".");
else
m_errorReporter.warning(_functionCall.location(), "Invoking events without \"emit\" prefix is deprecated.");
}
m_errorReporter.typeError(_functionCall.location(), "Event invocations have to be prefixed by \"emit\".");
TypePointers parameterTypes = functionType->parameterTypes();

3
test/libsolidity/syntaxTests/nameAndTypeResolution/080_event.sol
View File

@ -1,6 +1,5 @@
contract c {
event e(uint indexed a, bytes3 indexed s, bool indexed b);
function f() public { e(2, "abc", true); }
function f() public { emit e(2, "abc", true); }
}
// ----
// Warning: (102-119): Invoking events without "emit" prefix is deprecated.

3
test/libsolidity/syntaxTests/nameAndTypeResolution/090_event_call.sol
View File

@ -1,6 +1,5 @@
contract c {
event e(uint a, bytes3 indexed s, bool indexed b);
function f() public { e(2, "abc", true); }
function f() public { emit e(2, "abc", true); }
}
// ----
// Warning: (94-111): Invoking events without "emit" prefix is deprecated.

3
test/libsolidity/syntaxTests/nameAndTypeResolution/094_event_inheritance.sol
View File

@ -2,7 +2,6 @@ contract base {
event e(uint a, bytes3 indexed s, bool indexed b);
}
contract c is base {
function f() public { e(2, "abc", true); }
function f() public { emit e(2, "abc", true); }
}
// ----
// Warning: (120-137): Invoking events without "emit" prefix is deprecated.

2
test/libsolidity/syntaxTests/nameAndTypeResolution/563_event_without_emit_deprecated.sol
View File

@ -5,4 +5,4 @@ contract C {
}
}
// ----
// Warning: (62-65): Invoking events without "emit" prefix is deprecated.
// TypeError: (62-65): Event invocations have to be prefixed by "emit".

4
test/libsolidity/syntaxTests/types/empty_tuple_event.sol
View File

@ -2,9 +2,9 @@ pragma solidity ^0.4.3;
contract C {
event SomeEvent();
function a() public {
(SomeEvent(), 7);
(emit SomeEvent(), 7);
}
}
// ----
// Warning: (95-106): Invoking events without "emit" prefix is deprecated.
// TypeError: (95-106): Event invocations have to be prefixed by "emit".
// Warning: (95-106): Tuple component cannot be empty.

10
test/libsolidity/syntaxTests/types/empty_tuple_event_050.sol
View File

@ -1,10 +0,0 @@
pragma experimental "v0.5.0";
contract C {
event SomeEvent();
function a() public {
(SomeEvent(), 7);
}
}
// ----
// TypeError: (101-112): Event invocations have to be prefixed by "emit".
// TypeError: (101-112): Tuple component cannot be empty.