Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@cameel
Copy link
Collaborator

@cameel cameel commented Oct 27, 2021

On breaking inline assembly with name clashes is disallowed so the tests added in #12139 cause syntax errors.

This PR replaces command-line/semantic tests with syntax tests and updates their expectations.

@cameel cameel requested a review from chriseth October 27, 2021 17:49
@cameel cameel self-assigned this Oct 27, 2021
@cameel cameel changed the title Update expectation in tests for inline assembly with name clashes on breaking Update expectations in tests for inline assembly with name clashes on breaking Oct 27, 2021
@cameel cameel force-pushed the update-expectations-in-tests-with-inline-assembly-name-clashes branch from 3b50011 to 910ea59 Compare October 27, 2021 18:16
@cameel
Copy link
Collaborator Author

cameel commented Oct 27, 2021

There seems to be a problem with Windows VMs on CircleCI.

@chriseth chriseth merged commit f0d671c into breaking Oct 28, 2021
@chriseth chriseth deleted the update-expectations-in-tests-with-inline-assembly-name-clashes branch October 28, 2021 09:04
Marenz added a commit that referenced this pull request Aug 30, 2022
Manual Resolved Conflicts:
	Changelog.md
	 * Updated changelog
	test/externalTests/ens.sh
	 * Merged fixes for upstream from both develop and breaking
	test/libsolidity/semanticTests/inlineAssembly/external_identifier_access_shadowing.sol
	 * Removed in #11735 (breaking)
	test/libsolidity/semanticTests/inlineAssembly/function_name_clash.sol
	 * Removed in #12209 (breaking)
	test/libsolidity/semanticTests/storage/mappings_array2d_pop_delete.sol
	 * Removed in #11843 (breaking)
	test/libsolidity/semanticTests/storage/mappings_array_pop_delete.sol
	 * Removed in #11843 (breaking)
	test/libsolidity/syntaxTests/inlineAssembly/basefee_berlin_function.sol
	 * Used version of file from #11842 (breaking)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants