mirror of
https://github.com/ethereum/solidity
synced 2023-10-03 13:03:40 +00:00
Merge pull request #2163 from chriseth/sol_memoryArrays3
Pointers are not references.
This commit is contained in:
commit
fd74541843
@ -1889,6 +1889,93 @@ BOOST_AUTO_TEST_CASE(storage_location_local_variables)
|
|||||||
BOOST_CHECK_NO_THROW(parseTextAndResolveNames(sourceCode));
|
BOOST_CHECK_NO_THROW(parseTextAndResolveNames(sourceCode));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
BOOST_AUTO_TEST_CASE(assignment_mem_to_local_storage_variable)
|
||||||
|
{
|
||||||
|
char const* sourceCode = R"(
|
||||||
|
contract C {
|
||||||
|
uint[] data;
|
||||||
|
function f(uint[] x) {
|
||||||
|
var dataRef = data;
|
||||||
|
dataRef = x;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
)";
|
||||||
|
BOOST_CHECK_THROW(parseTextAndResolveNames(sourceCode), TypeError);
|
||||||
|
}
|
||||||
|
|
||||||
|
BOOST_AUTO_TEST_CASE(storage_assign_to_different_local_variable)
|
||||||
|
{
|
||||||
|
char const* sourceCode = R"(
|
||||||
|
contract C {
|
||||||
|
uint[] data;
|
||||||
|
uint8[] otherData;
|
||||||
|
function f() {
|
||||||
|
uint8[] storage x = otherData;
|
||||||
|
uint[] storage y = data;
|
||||||
|
y = x;
|
||||||
|
// note that data = otherData works
|
||||||
|
}
|
||||||
|
}
|
||||||
|
)";
|
||||||
|
BOOST_CHECK_THROW(parseTextAndResolveNames(sourceCode), TypeError);
|
||||||
|
}
|
||||||
|
|
||||||
|
BOOST_AUTO_TEST_CASE(assignment_mem_storage_variable_directly)
|
||||||
|
{
|
||||||
|
char const* sourceCode = R"(
|
||||||
|
contract C {
|
||||||
|
uint[] data;
|
||||||
|
function f(uint[] x) {
|
||||||
|
data = x;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
)";
|
||||||
|
BOOST_CHECK_NO_THROW(parseTextAndResolveNames(sourceCode));
|
||||||
|
}
|
||||||
|
|
||||||
|
BOOST_AUTO_TEST_CASE(function_argument_mem_to_storage)
|
||||||
|
{
|
||||||
|
char const* sourceCode = R"(
|
||||||
|
contract C {
|
||||||
|
function f(uint[] storage x) private {
|
||||||
|
}
|
||||||
|
function g(uint[] x) {
|
||||||
|
f(x);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
)";
|
||||||
|
BOOST_CHECK_THROW(parseTextAndResolveNames(sourceCode), TypeError);
|
||||||
|
}
|
||||||
|
|
||||||
|
BOOST_AUTO_TEST_CASE(function_argument_storage_to_mem)
|
||||||
|
{
|
||||||
|
char const* sourceCode = R"(
|
||||||
|
contract C {
|
||||||
|
function f(uint[] storage x) private {
|
||||||
|
g(x);
|
||||||
|
}
|
||||||
|
function g(uint[] x) {
|
||||||
|
}
|
||||||
|
}
|
||||||
|
)";
|
||||||
|
BOOST_CHECK_NO_THROW(parseTextAndResolveNames(sourceCode));
|
||||||
|
}
|
||||||
|
|
||||||
|
BOOST_AUTO_TEST_CASE(mem_array_assignment_changes_base_type)
|
||||||
|
{
|
||||||
|
// Such an assignment is possible in storage, but not in memory
|
||||||
|
// (because it would incur an otherwise unnecessary copy).
|
||||||
|
// This requirement might be lifted, though.
|
||||||
|
char const* sourceCode = R"(
|
||||||
|
contract C {
|
||||||
|
function f(uint8[] memory x) private {
|
||||||
|
uint[] memory y = x;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
)";
|
||||||
|
BOOST_CHECK_THROW(parseTextAndResolveNames(sourceCode), TypeError);
|
||||||
|
}
|
||||||
|
|
||||||
BOOST_AUTO_TEST_SUITE_END()
|
BOOST_AUTO_TEST_SUITE_END()
|
||||||
|
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user