Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Revert "Move clashing libraries to common source in test" | Rhett Aultman | 2017-01-17 | 1 | -1/+2 |
| | | | | This reverts commit c4a9ca5cfe7a8b4ba9d2d84392c57d5eefacd1f7. | ||||
* | Remove checkLibraryNameClashes() | Rhett Aultman | 2017-01-17 | 2 | -37/+0 |
| | | | | | | | | The library name clash checker throws errors when two libraries of the same name are spotted. In a previous commit, this function was rewritten to use fully-qualified names instead, which makes it redundant to the checker for multiply-declared identifiers. Since it no longer serves a clear purpose, the function is being dropped. | ||||
* | Use fully-qualified names for linking, too | Rhett Aultman | 2017-01-17 | 3 | -15/+15 |
| | | | | | | | | | | | | | | | Using libraries leaves behind a library link reference in the binary which the linker must later resolve. These link references were still being generated by name and not by fully-qualified name. This would lead to a link-time collision between two libraries having the same name but in different source units. This change changes linker symbols over to fully-qualified names, which resolves that issue. This does potentially introduce a new problem, which is that linker symbols appear to be limited to 36 characters and are truncated. Storing paths extends the average symbol size, and it would be great if truncation was from the tail rather than the head. | ||||
* | Remove unique error for contract collision | Rhett Aultman | 2017-01-17 | 1 | -39/+10 |
| | | | | | | Because contracts are uniquely identified by their source unit, there is no need for a unique error for this; it's actually covered by the checker for double-declaration of identifiers. | ||||
* | Fix contract lookup in tests | Rhett Aultman | 2017-01-17 | 5 | -6/+10 |
| | | | | | | The fully-qualified name of a contract with no source unit is :<Name> instead of just <Name>, so the test system needed to be adjusted accordingly. | ||||
* | Move fullyQualified() name to Declaration | Rhett Aultman | 2017-01-17 | 2 | -10/+1 |
| | |||||
* | Stylistic corrections | Rhett Aultman | 2017-01-17 | 3 | -11/+5 |
| | |||||
* | Move clashing libraries to common source in test | Rhett Aultman | 2017-01-17 | 1 | -2/+1 |
| | | | | | | Since contracts and libraries only collide if they share a common source file now, this test only works if both libraries are in the same source. | ||||
* | Push the error instead of throwing it | Rhett Aultman | 2017-01-17 | 1 | -10/+34 |
| | | | | | | | Throwing a CompilerError on multiple contract definition violates the expectations of the test suite, which thinks that compile() will return false if the code can't compile. This brings contract collision reporting in line with most of the other errors. | ||||
* | Drop ':' if the source file name is empty | Rhett Aultman | 2017-01-17 | 1 | -1/+2 |
| | | | | | | | | A large number of tests compile contracts while passing in an empty string for the source name. This leads to it being keyed by the name ":<contract>", while the tests try to look it up under the name "<contract>". This change resolves that issue by dropping the ':' in cases where there is, effectively, no source file to prepend anyway. | ||||
* | Only avoid collision if it's the same file | Rhett Aultman | 2017-01-17 | 5 | -14/+52 |
| | | | | | | | | | @chriseth had suggested that it would be better if contracts were referenced in a file:contract notation, and that we output .bin files that prepend original path names if necessary to avoid a collision. This commit is mostly a draft; it still needs to be run through the test suite. | ||||
* | Tab whitespace cleanup (again) | Rhett Aultman | 2017-01-17 | 1 | -1/+2 |
| | |||||
* | Tidy up the error message | Rhett Aultman | 2017-01-17 | 1 | -3/+3 |
| | |||||
* | Fix tab, drop stupid '!!!', change error message. | Rhett Aultman | 2017-01-17 | 1 | -3/+3 |
| | |||||
* | Error out when contracts collide on name | Rhett Aultman | 2017-01-17 | 1 | -0/+19 |
| | | | | | | | | | | | | | | The previous behaviour, courtesy of the [] operator in std::map, would uncritically store a new ContractDefinition in m_contracts even when a ContractDefinition already existed. This "resolved" collissions on contract names by clobbering the original one with the new one, and could lead to scenarios where the clobber would only be discovered when the original ContractDefinition could not be found or referred to, which was an unhelpful InternalCompilerError. This change checks the m_contracts map for a collision first and will not let the ContractDefinition be changed to a new one once it's set, throwing a CompilerError with information about the conflict. | ||||
* | Update SolidityLexer.py (#1567) | Nicola | 2017-01-16 | 1 | -0/+1 |
| | | | Added number unit keywords | ||||
* | Merge pull request #1564 from natzei/patch-1 | chriseth | 2017-01-13 | 1 | -1/+1 |
|\ | | | | | Update SolidityLexer.py | ||||
| * | Update SolidityLexer.py | Nicola | 2017-01-13 | 1 | -1/+1 |
|/ | | | 'payable' added as a keyword | ||||
* | Merge pull request #1562 from ethereum/gas-tier-enum-class | chriseth | 2017-01-13 | 3 | -141/+141 |
|\ | | | | | Make enum Tier into an enum class | ||||
| * | Make enum Tier into an enum class | Yoichi Hirai | 2017-01-13 | 3 | -141/+141 |
| | | |||||
* | | Merge pull request #1563 from ethereum/updateVersion | chriseth | 2017-01-13 | 3 | -3/+5 |
|\ \ | |/ |/| | Update version to 0.4.9. | ||||
| * | Update version to 0.4.9. | chriseth | 2017-01-13 | 3 | -3/+5 |
|/ | |||||
* | Update Changelog.md | chriseth | 2017-01-13 | 1 | -1/+1 |
| | |||||
* | Merge pull request #1479 from ethereum/function_variable_mixin | chriseth | 2017-01-13 | 9 | -67/+190 |
|\ | | | | | Disallow mixin of functions and attributes under the same name | ||||
| * | ast: events have FunctionType too | Yoichi Hirai | 2017-01-12 | 2 | -8/+17 |
| | | |||||
| * | analysis: use Declaration::functionType() in another location | Yoichi Hirai | 2017-01-12 | 1 | -11/+4 |
| | | |||||
| * | ast: add Declaration::functionType() | Yoichi Hirai | 2017-01-12 | 3 | -14/+80 |
| | | |||||
| * | Changelog: add a line about #1479 | Yoichi Hirai | 2017-01-12 | 1 | -0/+1 |
| | | |||||
| * | docs: document the resriction about a function and an event of the same ↵ | Yoichi Hirai | 2017-01-12 | 1 | -0/+7 |
| | | | | | | | | name, and so on | ||||
| * | test: allow multiple errors in modifier-function overriding | Yoichi Hirai | 2017-01-12 | 1 | -2/+6 |
| | | |||||
| * | analysis: avoid emscripten build failure | Yoichi Hirai | 2017-01-12 | 1 | -3/+3 |
| | | |||||
| * | analysis: changes necessary to compile std/StandardToken.sol | Yoichi Hirai | 2017-01-12 | 3 | -12/+58 |
| | | |||||
| * | analysis: allow some shadowings explicitly | Yoichi Hirai | 2017-01-12 | 3 | -56/+29 |
| | | |||||
| * | analysis: report errors when inheritance causes collision | Yoichi Hirai | 2017-01-12 | 1 | -1/+25 |
| | | |||||
* | | Merge pull request #1560 from ethereum/cmake | chriseth | 2017-01-13 | 2 | -0/+39 |
|\ \ | | | | | | | Travis CI: Install latest CMake | ||||
| * | | Travis CI: Install latest CMake | Paweł Bylica | 2017-01-12 | 2 | -0/+39 |
| | | | |||||
* | | | Merge pull request #1549 from ethereum/optimizeOptimizer | chriseth | 2017-01-13 | 12 | -461/+646 |
|\ \ \ | |/ / |/| | | Optimize optimizer | ||||
| * | | Minor changes from review. | chriseth | 2017-01-13 | 4 | -4/+3 |
| | | | |||||
| * | | Changelog entry. | chriseth | 2017-01-13 | 1 | -1/+2 |
| | | | |||||
| * | | Windows build fix. | chriseth | 2017-01-13 | 1 | -0/+2 |
| | | | |||||
| * | | Fix match groups. | chriseth | 2017-01-13 | 1 | -0/+1 |
| | | | |||||
| * | | Fix debug output. | chriseth | 2017-01-13 | 1 | -2/+8 |
| | | | |||||
| * | | Optimize constant optimization. | chriseth | 2017-01-13 | 2 | -37/+58 |
| | | | |||||
| * | | Optimise simplification rules (sort into bins). | chriseth | 2017-01-13 | 4 | -394/+514 |
| | | | |||||
| * | | Do not evaluate strings if assertion succeeds. | chriseth | 2017-01-13 | 1 | -20/+12 |
| | | | |||||
| * | | Optimize pattern matcher. | chriseth | 2017-01-13 | 2 | -16/+22 |
| | | | |||||
| * | | Optimise AssemblyItem::m_data. | chriseth | 2017-01-13 | 5 | -22/+59 |
|/ / | |||||
* | | Merge pull request #1548 from VoR0220/remappingBugFix | chriseth | 2017-01-12 | 4 | -9/+51 |
|\ \ | | | | | | | Remapping bug fix | ||||
| * | | fixed | VoR0220 | 2017-01-12 | 2 | -4/+4 |
| | | | | | | | | | | | | Signed-off-by: VoR0220 <rj@erisindustries.com> | ||||
| * | | added fix and a test for order independence of nested prefixing | VoR0220 | 2017-01-12 | 2 | -2/+20 |
| | | | | | | | | | | | | Signed-off-by: VoR0220 <rj@erisindustries.com> | ||||
| * | | cleanup | VoR0220 | 2017-01-12 | 3 | -16/+16 |
| | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: VoR0220 <rj@erisindustries.com> fixup Signed-off-by: VoR0220 <rj@erisindustries.com> | ||||
| * | | changelog entry | VoR0220 | 2017-01-11 | 1 | -0/+2 |
| | | | | | | | | | | | | Signed-off-by: VoR0220 <rj@erisindustries.com> | ||||
| * | | fixed unused filepath bug | VoR0220 | 2017-01-11 | 1 | -2/+1 |
| | | | | | | | | | | | | Signed-off-by: VoR0220 <rj@erisindustries.com> | ||||
| * | | much smaller helper function | VoR0220 | 2017-01-10 | 2 | -5/+5 |
| | | | | | | | | | | | | Signed-off-by: VoR0220 <rj@erisindustries.com> | ||||
| * | | can do this purely on length. Also made prefix filesystem string for more ↵ | VoR0220 | 2017-01-09 | 1 | -27/+16 |
| | | | | | | | | | | | | | | | | | | accurate readings. Signed-off-by: VoR0220 <rj@erisindustries.com> | ||||
| * | | fixed test and added solution | VoR0220 | 2017-01-09 | 2 | -5/+28 |
| | | | | | | | | | | | | Signed-off-by: VoR0220 <rj@erisindustries.com> | ||||
| * | | added test | VoR0220 | 2017-01-09 | 1 | -0/+11 |
| | | | | | | | | | | | | Signed-off-by: VoR0220 <rj@erisindustries.com> | ||||
* | | | Merge pull request #1558 from anders94/doc-rewording | chriseth | 2017-01-12 | 1 | -2/+2 |
|\ \ \ | |_|/ |/| | | Re-wording for Clarity | ||||
| * | | Re-word somewhat bumpy sentence to be more clear. | Anders Brownworth | 2017-01-12 | 1 | -2/+2 |
| | | | |||||
* | | | Merge pull request #1555 from ethereum/chriseth-patch-1 | Yoichi Hirai | 2017-01-12 | 1 | -0/+3 |
|\ \ \ | |/ / |/| | | Changelog entry about assembly output | ||||
| * | | Changelog entry about assembly output | chriseth | 2017-01-11 | 1 | -0/+3 |
|/ / | |||||
* | | Merge pull request #1553 from federicobond/improve-grammar | chriseth | 2017-01-11 | 1 | -5/+11 |
|\ \ | | | | | | | Improve grammar specification | ||||
| * | | grammar.txt: Replace Identifier for Typename in NewExpression | Federico Bond | 2017-01-11 | 1 | -1/+1 |
| | | | |||||
| * | | grammar.txt: Add named function arguments | Federico Bond | 2017-01-11 | 1 | -1/+7 |
| | | | |||||
| * | | grammar.txt: Move StorageLocation? up to VariableDeclaration | Federico Bond | 2017-01-11 | 1 | -3/+3 |
|/ / | |||||
* | | Merge pull request #1330 from ethereum/assemblyDef | chriseth | 2017-01-11 | 5 | -51/+1056 |
|\ \ | | | | | | | Assembly definition. | ||||
| * | | Move explanatory sections and other small changes. | chriseth | 2017-01-09 | 1 | -183/+192 |
| | | | |||||
| * | | Typos and clarifications. | chriseth | 2017-01-04 | 1 | -13/+17 |
| | | | |||||
| * | | More information about switch, loops and functions. | chriseth | 2017-01-04 | 1 | -1/+78 |
| | | | |||||
| * | | Update to new assembly specification. | chriseth | 2017-01-03 | 1 | -31/+381 |
| | | | |||||
| * | | Rename programSize to bytecodeSize. | chriseth | 2017-01-03 | 1 | -1/+1 |
| | | | |||||
| * | | Assembly output for Assembly object. | chriseth | 2017-01-03 | 3 | -51/+105 |
| | | | |||||
| * | | Assembly definition. | chriseth | 2017-01-03 | 2 | -0/+511 |
| | | | |||||
* | | | Merge pull request #1542 from ethereum/publishTarball | chriseth | 2017-01-09 | 2 | -36/+49 |
|\ \ \ | |_|/ |/| | | Create source tarballs for releases. | ||||
| * | | Replace `` by $(). | chriseth | 2017-01-09 | 1 | -4/+3 |
| | | | |||||
| * | | Create source tarballs for releases. | chriseth | 2017-01-04 | 2 | -36/+50 |
| | | | |||||
* | | | Merge pull request #1537 from ethereum/absolute-path | chriseth | 2017-01-07 | 4 | -5/+15 |
|\ \ \ | | | | | | | | | absolute path detection is not confused by ".dir" | ||||
| * | | | Changelog: document #1537 | Yoichi Hirai | 2017-01-06 | 1 | -0/+1 |
| | | | | |||||
| * | | | test: add a test importing a file name starting with a period | Yoichi Hirai | 2017-01-04 | 1 | -0/+8 |
| | | | | |||||
| * | | | interface: change absolutePath() so that ".dir" is considered as an absolute ↵ | Yoichi Hirai | 2017-01-04 | 1 | -3/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | path fixes #1534 | ||||
| * | | | docs: clarify what are considered as absolute paths | Yoichi Hirai | 2017-01-04 | 1 | -2/+3 |
| |/ / | |||||
* | | | Merge pull request #1545 from federicobond/refactor-rename | chriseth | 2017-01-05 | 1 | -3/+3 |
|\ \ \ | |/ / |/| | | Rename contractName to typeName when parsing new expression | ||||
| * | | Rename contractName to typeName when parsing new expression | Federico Bond | 2017-01-05 | 1 | -3/+3 |
| | | | |||||
* | | | Merge pull request #1541 from ethereum/stabelnodejs | chriseth | 2017-01-03 | 1 | -0/+1 |
|\ \ \ | |_|/ |/| | | Require stable nodejs for deployment. | ||||
| * | | Require stable nodejs for deployment. | chriseth | 2017-01-03 | 1 | -0/+1 |
| | | | |||||
* | | | Merge pull request #1474 from ethereum/event-lookup-base-name | chriseth | 2017-01-03 | 4 | -9/+61 |
|\ \ \ | | | | | | | | | Event lookup with base name | ||||
| * | | | ast, codegen: enable accessing events through contract names. | Yoichi Hirai | 2017-01-03 | 3 | -9/+39 |
| | | | | | | | | | | | | | | | | Fixes #1286 | ||||
| * | | | test: add a test case for #1286 | Yoichi Hirai | 2017-01-03 | 1 | -0/+22 |
| | | | | |||||
* | | | | Merge pull request #1527 from federicobond/elementarytypenameconversion | chriseth | 2017-01-03 | 1 | -1/+8 |
|\ \ \ \ | | | | | | | | | | | Add ElementaryTypeNameExpression to grammar.txt | ||||
| * | | | | Add ElementaryTypeNameExpression to grammar.txt | Federico Bond | 2017-01-02 | 1 | -1/+8 |
| | |_|/ | |/| | | |||||
* | | | | Merge pull request #1538 from ethereum/multi-keys | chriseth | 2017-01-03 | 1 | -2/+2 |
|\ \ \ \ | | | | | | | | | | | doc: rephrase the case when two remapping keys are applicable | ||||
| * | | | | doc: rephrase the case when two remapping keys are applicable | Yoichi Hirai | 2017-01-02 | 1 | -2/+2 |
| | |/ / | |/| | | | | | | | | | | Fixes #1536 | ||||
* | | | | Merge pull request #1526 from federicobond/docs-grammar | chriseth | 2017-01-03 | 1 | -0/+9 |
|\ \ \ \ | |/ / / |/| | | | Add parsers and grammars to documentation | ||||
| * | | | Add parsers and grammars to documentation | Federico Bond | 2017-01-02 | 1 | -0/+9 |
| | | | | |||||
* | | | | Merge pull request #1530 from federicobond/remove-syntax-highlighting | chriseth | 2017-01-02 | 1 | -0/+1 |
|\ \ \ \ | | | | | | | | | | | Remove wrong syntax highlighting in docs | ||||
| * | | | | Remove wrong syntax highlighting in docs | Federico Bond | 2016-12-29 | 1 | -0/+1 |
| | |/ / | |/| | | |||||
* | | | | Merge pull request #1529 from federicobond/right-shifts | chriseth | 2017-01-02 | 1 | -1/+1 |
|\ \ \ \ | | | | | | | | | | | Add missing right-hand side expression for shifts | ||||
| * | | | | Add missing right-hand side expression for shifts | Federico Bond | 2016-12-28 | 1 | -1/+1 |
| |/ / / | |||||
* | | | | Merge pull request #1528 from federicobond/userdefinedtypename | chriseth | 2017-01-02 | 1 | -2/+10 |
|\ \ \ \ | | | | | | | | | | | Add UserDefinedTypeName to grammar.txt | ||||
| * | | | | Add UserDefinedTypeName to grammar.txt | Federico Bond | 2016-12-28 | 1 | -2/+10 |
| |/ / / | |||||
* | | | | Merge pull request #1531 from ethereum/srcmap | chriseth | 2017-01-02 | 1 | -1/+1 |
|\ \ \ \ | |/ / / |/| | | | libevmasm: OpPop() optimization keeps the source location | ||||
| * | | | libevmasm: OpPop() optimization keeps the source location | Yoichi Hirai | 2016-12-30 | 1 | -1/+1 |
|/ / / | | | | | | | | | | fixes #1521 | ||||
* | | | Merge pull request #1522 from federicobond/patch-2 | Yoichi Hirai | 2016-12-27 | 1 | -3/+6 |
|\ \ \ | | | | | | | | | Fix some grammar.txt inconsistencies | ||||
| * | | | Add unimplemented functions to grammar | Federico Bond | 2016-12-27 | 1 | -1/+1 |
| | | | | |||||
| * | | | Fix NumberLiteral rule for hex numbers | Federico Bond | 2016-12-25 | 1 | -1/+4 |
| | | | | |||||
| * | | | Fix StorageLocation position for ArrayTypeName rule in grammar | Federico Bond | 2016-12-25 | 1 | -1/+1 |
|/ / / | |||||
* | | | Merge pull request #1518 from mcoblenz/patch-2 | chriseth | 2016-12-22 | 1 | -2/+2 |
|\ \ \ | | | | | | | | | Corrected explanation of recursive contract creation | ||||
| * | | | Corrected explanation per discussion. | Michael Coblenz | 2016-12-22 | 1 | -2/+2 |
| | | | | |||||
| * | | | Clarified support for recursive creation. | Michael Coblenz | 2016-12-21 | 1 | -2/+2 |
| | | | | |||||
| * | | | Fixed misleading typo | Michael Coblenz | 2016-12-20 | 1 | -1/+1 |
| | |/ | |/| | | | | "now possible" should instead read "not possible." | ||||
* | | | Merge pull request #1519 from federicobond/patch-1 | Yoichi Hirai | 2016-12-22 | 1 | -1/+1 |
|\ \ \ | |/ / |/| | | Fix mapping syntax in docs | ||||
| * | | Fix mapping syntax in docs | Federico Bond | 2016-12-22 | 1 | -1/+1 |
|/ / | |||||
* | | Merge pull request #1515 from ethereum/version | chriseth | 2016-12-16 | 3 | -3/+5 |
|\ \ | |/ |/| | Update develop version. | ||||
| * | Update develop version. | chriseth | 2016-12-15 | 3 | -3/+5 |
|/ | |||||
* | Merge pull request #1514 from ethereum/changelog | chriseth | 2016-12-15 | 1 | -4/+4 |
|\ | | | | | Update changelog for release. | ||||
| * | Update changelog for release. | chriseth | 2016-12-15 | 1 | -4/+4 |
|/ | |||||
* | Merge pull request #1513 from ethereum/changelog-for-ast-json | chriseth | 2016-12-15 | 1 | -0/+3 |
|\ | | | | | Changelog: add entries for #1497 and #1503 | ||||
| * | Changelog: add entries for #1497 and #1503 | Yoichi Hirai | 2016-12-15 | 1 | -0/+3 |
|/ | |||||
* | Merge pull request #1503 from federicobond/json-visibility | Yoichi Hirai | 2016-12-15 | 2 | -7/+22 |
|\ | | | | | Include visibility level for functions in AST | ||||
| * | Include visibility level for functions in AST | Federico Bond | 2016-12-13 | 2 | -7/+22 |
| | | |||||
* | | Merge pull request #1508 from ethereum/multiurls | chriseth | 2016-12-15 | 2 | -6/+8 |
|\ \ | | | | | | | Use multiple URLs for metadata. | ||||
| * | | Use multiple URLs for metadata. | chriseth | 2016-12-13 | 2 | -6/+8 |
| | | | |||||
* | | | Merge pull request #1487 from ethereum/shift-ops | Yoichi Hirai | 2016-12-14 | 7 | -42/+543 |
|\ \ \ | | | | | | | | | Shift operators | ||||
| * | | | Fix tests. | chriseth | 2016-12-14 | 1 | -5/+5 |
| | | | | |||||
| * | | | Use correct type for storing. | chriseth | 2016-12-13 | 1 | -7/+7 |
| | | | | |||||
| * | | | Update documentation. | chriseth | 2016-12-13 | 1 | -7/+7 |
| | | | | |||||
| * | | | test: add tests that tries different types on <<= | Yoichi Hirai | 2016-12-13 | 1 | -0/+18 |
| | | | | |||||
| * | | | Documentation. | chriseth | 2016-12-12 | 2 | -11/+21 |
| | | | | |||||
| * | | | Tests for bytes. | chriseth | 2016-12-12 | 1 | -0/+38 |
| | | | | |||||
| * | | | Correct test expectations. | chriseth | 2016-12-12 | 1 | -3/+5 |
| | | | | |||||
| * | | | Display tx hash for debugging. | chriseth | 2016-12-12 | 1 | -2/+5 |
| | | | | |||||
| * | | | Cleaner shift handling and type conversion for binary operations. | chriseth | 2016-12-12 | 4 | -78/+122 |
| | | | | |||||
| * | | | Type after shift should be type of left operand. | chriseth | 2016-12-12 | 2 | -5/+57 |
| | | | | |||||
| * | | | Support bitshifting in variables | Alex Beregszaszi | 2016-12-12 | 5 | -9/+346 |
| | | | | |||||
* | | | | Merge pull request #1507 from ethereum/memcopystyle | Yoichi Hirai | 2016-12-14 | 2 | -14/+14 |
|\ \ \ \ | |_|/ / |/| | | | Renamed padToWordBoundaries -> padToWords | ||||
| * | | | Renamed padToWordBoundaries -> padToWords | chriseth | 2016-12-13 | 2 | -14/+14 |
| | | | | |||||
* | | | | Merge pull request #1506 from federicobond/improve-error-msg | chriseth | 2016-12-13 | 3 | -1/+19 |
|\ \ \ \ | |/ / / |/| | | | Improve error message when trying to modify constant variables | ||||
| * | | | Improve error message when trying to modify constant variables | Federico Bond | 2016-12-13 | 3 | -1/+19 |
|/ / / | |||||
* | | | Merge pull request #1468 from ethereum/memcpy-assembly | Yoichi Hirai | 2016-12-13 | 5 | -16/+83 |
|\ \ \ | | | | | | | | | Implement memcpy without the identity precompile | ||||
| * | | | Changelog entry. | chriseth | 2016-12-12 | 1 | -0/+1 |
| | | | | |||||
| * | | | Split memcopy into three functions. | chriseth | 2016-12-12 | 3 | -54/+79 |
| | | | | |||||
| * | | | Fix inline assembly. | chriseth | 2016-12-12 | 1 | -1/+2 |
| | | | | |||||
| * | | | Reset stack height after inline assembly block | Alex Beregszaszi | 2016-12-12 | 1 | -0/+1 |
| | | | | |||||
| * | | | Better assert message for appendInlineAssembnly | Alex Beregszaszi | 2016-12-12 | 1 | -1/+1 |
| | | | | |||||
| * | | | Implement identity call in inline assembly | Alex Beregszaszi | 2016-12-12 | 1 | -14/+21 |
| | | | | |||||
| * | | | Turn off the identity precompile for testing | Alex Beregszaszi | 2016-12-12 | 1 | -1/+1 |
| | | | | |||||
| * | | | Implement CompilerUtils::memoryCopy using inline assembly | Alex Beregszaszi | 2016-12-12 | 1 | -1/+26 |
| | | | | |||||
| * | | | Add usingIdentity option to CompilerUtils::memoryCopy | Alex Beregszaszi | 2016-12-12 | 2 | -2/+9 |
| |/ / | |||||
* | | | Merge pull request #1491 from ameten/refactoring | chriseth | 2016-12-13 | 2 | -139/+184 |
|\ \ \ | | | | | | | | | Just refactored some code | ||||
| * | | | - alphabetically sorted string constants per name. | Danil Nemirovsky | 2016-12-09 | 1 | -45/+45 |
| | | | | |||||
| * | | | - moved string constants into common place; | Danil Nemirovsky | 2016-12-09 | 1 | -136/+181 |
| | | | | | | | | | | | | | | | | - replaced string literals with string constants. | ||||
| * | | | - fixed a misprint. | Danil Nemirovsky | 2016-12-09 | 1 | -1/+1 |
| | | | | |||||
* | | | | Merge pull request #1494 from ethereum/showtxhash | Yoichi Hirai | 2016-12-13 | 1 | -2/+5 |
|\ \ \ \ | |_|_|/ |/| | | | Display tx hash for debugging. | ||||
| * | | | Display tx hash for debugging. | chriseth | 2016-12-10 | 1 | -2/+5 |
| |/ / | |||||
* | | | Merge pull request #1357 from federicobond/remove-sierra-note | chriseth | 2016-12-12 | 1 | -6/+0 |
|\ \ \ | |_|/ |/| | | Remove macOS Sierra note from install_deps.sh | ||||
| * | | Remove macOS Sierra note from install_deps.sh | Federico Bond | 2016-11-11 | 1 | -6/+0 |
| | | | |||||
* | | | Warn if msg.value is used in non-payable function. | chriseth | 2016-12-11 | 1 | -0/+1 |
| | | | |||||
* | | | Merge pull request #1464 from federicobond/warn-msgvalue | chriseth | 2016-12-11 | 5 | -1/+251 |
|\ \ \ | | | | | | | | | Warn about using msg.value in non-payable function | ||||
| * | | | Warn about using msg.value in non-payable function | Federico Bond | 2016-12-09 | 5 | -1/+251 |
| | |/ | |/| | |||||
* | | | Merge pull request #1481 from zigguratt/lll-end-to-end-testing | chriseth | 2016-12-11 | 1 | -0/+229 |
|\ \ \ | | | | | | | | | LLL: end to end testing | ||||
| * | | | Removed 'panic' from 'constructor_arguments_external' and added specific | Daniel Ellison | 2016-12-07 | 1 | -2/+147 |
| | | | | | | | | | | | | | | | | | | | | | | | | test for fallback and fallthrough. Added one test for 'lit'. Added tests for arithmetic/binary/unary operations. Added the start of tests for assembler instructions within LLL source. | ||||
| * | | | Added tests for LLL constructors. Fixed a few tab issues. | Daniel Ellison | 2016-12-07 | 1 | -5/+49 |
| | | | | |||||
| * | | | Removed enclosed_panic test until assembler bug is fixed. | Daniel Ellison | 2016-12-07 | 1 | -7/+0 |
| | | | | |||||
| * | | | Simplified exp_operator_on_range test and changed to tedting a range. | Daniel Ellison | 2016-12-07 | 1 | -11/+7 |
| | | | | |||||
| * | | | Added tests for exponent operator. | Daniel Ellison | 2016-12-07 | 1 | -0/+37 |
| | | | | |||||
| * | | | Added 'panic' tests. | Daniel Ellison | 2016-12-07 | 1 | -0/+14 |
| | | | | |||||
* | | | | Merge pull request #1496 from federicobond/remove-unused-var | chriseth | 2016-12-11 | 1 | -1/+0 |
|\ \ \ \ | | | | | | | | | | | Remove unused variable inputFiles | ||||
| * | | | | Remove unused variable inputFiles | Federico Bond | 2016-12-11 | 1 | -1/+0 |
| | |/ / | |/| | | |||||
* | | | | Merge pull request #1497 from federicobond/json-names | chriseth | 2016-12-11 | 1 | -2/+2 |
|\ \ \ \ | | | | | | | | | | | Fix names of JSON AST nodes | ||||
| * | | | | Fix names of JSON AST nodes | Federico Bond | 2016-12-11 | 1 | -2/+2 |
| |/ / / | |||||
* | | | | Merge pull request #1498 from federicobond/fix-typos | chriseth | 2016-12-11 | 1 | -2/+2 |
|\ \ \ \ | |/ / / |/| | | | Fix typos in comments | ||||
| * | | | Fix typos in comments | Federico Bond | 2016-12-11 | 1 | -2/+2 |
|/ / / | |||||
* | | | Merge pull request #1351 from ethereum/truncate_bit | chriseth | 2016-12-08 | 7 | -5/+89 |
|\ \ \ | | | | | | | | | Truncate a boolean from calldata into one bit | ||||
| * | | | docs: rename overflow to variable cleanup | Alex Beregszaszi | 2016-12-01 | 1 | -4/+4 |
| | | | | |||||
| * | | | docs: update overflow cleanup wording | Alex Beregszaszi | 2016-12-01 | 1 | -4/+4 |
| | | | | |||||
| * | | | codegen: clean not only booleans but all types before storing them into memory | Yoichi Hirai | 2016-12-01 | 2 | -4/+3 |
| | | | | |||||
| * | | | docs: remove the word overflow when we are talking about invalid values | Yoichi Hirai | 2016-12-01 | 1 | -22/+22 |
| | | | | |||||
| * | | | codegen: clean any data from the input | Yoichi Hirai | 2016-12-01 | 3 | -3/+6 |
| | | | | |||||
| * | | | docs: describe when and how overflown values are cleaned | Yoichi Hirai | 2016-12-01 | 2 | -2/+52 |
| | | | | |||||
| * | | | codegen: truncate booleans before they enter storage | Yoichi Hirai | 2016-12-01 | 1 | -1/+0 |
| | | | | |||||
| * | | | codegen: cleanup booleans before storing them into memory | Yoichi Hirai | 2016-12-01 | 1 | -0/+2 |
| | | | | |||||
| * | | | codegen: truncate a boolean calldata down to one bit | Yoichi Hirai | 2016-12-01 | 2 | -0/+3 |
| | | | | |||||
| * | | | test: add a test that witnesses #1318 | Yoichi Hirai | 2016-12-01 | 1 | -0/+28 |
| | | | | |||||
* | | | | Merge pull request #1412 from ethereum/disallow-multiple-errors | chriseth | 2016-12-08 | 1 | -25/+113 |
|\ \ \ \ | | | | | | | | | | | disallow more than one errors in name and type tests | ||||
| * | | | | test: either fix tests to raise just one error, or mark them as raising ↵ | Yoichi Hirai | 2016-12-05 | 1 | -32/+112 |
| | | | | | | | | | | | | | | | | | | | | multiple errors | ||||
| * | | | | test: disallow more than one errors in name and type tests | Yoichi Hirai | 2016-12-05 | 1 | -0/+8 |
| | | | | | |||||
* | | | | | Merge pull request #1488 from ethereum/parser-error-msg | chriseth | 2016-12-08 | 2 | -24/+49 |
|\ \ \ \ \ | | | | | | | | | | | | | test: check error messages in parser tests | ||||
| * | | | | | test: check error messages in parser tests | Yoichi Hirai | 2016-12-07 | 2 | -24/+49 |
| |/ / / / | |||||
* | | | | | Merge pull request #1492 from federicobond/patch-1 | chriseth | 2016-12-08 | 1 | -1/+1 |
|\ \ \ \ \ | | | | | | | | | | | | | Remove duplicate Identifier from AssemblyItem grammar rule | ||||
| * | | | | | Remove duplicate Identifier from AssemblyItem grammar rule | Federico Bond | 2016-12-08 | 1 | -1/+1 |
|/ / / / / | |||||
* | | | | | Merge pull request #1489 from federicobond/isolate-tests | chriseth | 2016-12-07 | 2 | -24/+44 |
|\ \ \ \ \ | | | | | | | | | | | | | Cleanup and fix scripts/isolate_tests.py | ||||
| * | | | | | Cleanup and fix scripts/isolate_tests.py | Federico Bond | 2016-12-07 | 1 | -16/+36 |
| | | | | | | |||||
| * | | | | | Rename isolateTests.py script to isolate_tests.py | Federico Bond | 2016-12-07 | 1 | -0/+0 |
| | | | | | | |||||
* | | | | | | Merge pull request #1490 from ethereum/debugMessages | chriseth | 2016-12-07 | 4 | -0/+18 |
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | Option to show messages in tests. | ||||
| * | | | | | Also provide value and format slightly differently. | chriseth | 2016-12-07 | 1 | -3/+6 |
| | | | | | | |||||
| * | | | | | Option to show messages in tests. | chriseth | 2016-12-07 | 4 | -0/+15 |
|/ / / / / | |||||
* | | | | | Merge pull request #1465 from ethereum/missing_keywords | chriseth | 2016-12-06 | 3 | -5/+13 |
|\ \ \ \ \ | |_|/ / / |/| | | | | document missing keywords | ||||
| * | | | | grammar.txt: define inline assembly | Yoichi Hirai | 2016-12-02 | 1 | -1/+9 |
| | | | | | |||||
| * | | | | parsing: `default` and `null` are reserved keywords | Yoichi Hirai | 2016-12-02 | 2 | -3/+3 |
| | | | | | |||||
| * | | | | grammar.txt: remove a space from `payable` keyword | Yoichi Hirai | 2016-12-02 | 1 | -1/+1 |
| | | | | | |||||
* | | | | | Merge pull request #1394 from ethers/patch-1 | chriseth | 2016-12-05 | 1 | -0/+81 |
|\ \ \ \ \ | |_|/ / / |/| | | | | styleguide: Order of Functions | ||||
| * | | | | Clearer language as suggested by @axic | ethers | 2016-11-22 | 1 | -1/+1 |
| | | | | | |||||
| * | | | | Fallback functions don't always have to be payable | ethers | 2016-11-18 | 1 | -2/+2 |
| | | | | | |||||
| * | | | | styleguide: Don't include a whitespace in the fallback function | ethers | 2016-11-18 | 1 | -0/+13 |
| | | | | | |||||
| * | | | | Don't include a whitespace in fallback function | ethers | 2016-11-18 | 1 | -2/+2 |
| | | | | | |||||
| * | | | | styleguide: Ordering of functions | ethers | 2016-11-18 | 1 | -0/+68 |
| | | | | | | | | | | | | | | | | | | | | | | | | | Ordering would help readers identify which functions they can call, and to find the "specials" (constructor and fallback function). Mixing the "specials" in the middle of the code, as well as internal functions between external and public functions, don't help readers Based on https://github.com/ConsenSys/MultiSigWallet/issues/19 | ||||
* | | | | | Merge pull request #1483 from federicobond/r-literals | RJ | 2016-12-04 | 4 | -592/+810 |
|\ \ \ \ \ | | | | | | | | | | | | | Migrate remaining source code in tests to R literals | ||||
| * | | | | | Migrate remaining source code in tests to R literals | Federico Bond | 2016-12-04 | 4 | -592/+810 |
|/ / / / / | |||||
* | | | | | Merge pull request #1475 from ethereum/fail-on-identity-fail | chriseth | 2016-12-03 | 2 | -1/+3 |
|\ \ \ \ \ | |_|/ / / |/| | | | | Throw if calling the identity precompile (memoryCopy) failed | ||||
| * | | | | Throw if calling the identity precompile (memoryCopy) failed | Alex Beregszaszi | 2016-12-02 | 2 | -1/+3 |
|/ / / / | |||||
* | | | | Merge pull request #1386 from ethereum/metadataOut | chriseth | 2016-12-02 | 26 | -116/+402 |
|\ \ \ \ | | | | | | | | | | | Metadata stamp | ||||
| * | | | | Count instructions without metadata hash. | chriseth | 2016-12-01 | 1 | -10/+22 |
| | | | | | |||||
| * | | | | Add the `_runs` parameter. | chriseth | 2016-12-01 | 2 | -3/+3 |
| | | | | | |||||
| * | | | | Submit library addresses to the compiler in tests (and not link afterwards) | Alex Beregszaszi | 2016-12-01 | 1 | -2/+1 |
| | | | | | |||||
| * | | | | Remove unneeded optimised test from EndToEndTest | Alex Beregszaszi | 2016-12-01 | 1 | -7/+0 |
| | | | | | |||||
| * | | | | Optimizer tests to not interfere with the optimiser flag | Alex Beregszaszi | 2016-12-01 | 1 | -11/+24 |
| | | | | | |||||
| * | | | | Fix type of optimizeRuns | Alex Beregszaszi | 2016-12-01 | 2 | -2/+2 |
| | | | | | |||||
| * | | | | Fix documentation errors | Alex Beregszaszi | 2016-12-01 | 1 | -4/+4 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Subheadings must use consistent format across a file. The metadata with comments cannot be parsed as JSON. | ||||
| * | | | | Fix tests | chriseth | 2016-12-01 | 1 | -2/+5 |
| | | | | | |||||
| * | | | | Update metadata documentation | Alex Beregszaszi | 2016-12-01 | 3 | -24/+26 |
| | | | | | |||||
| * | | | | Use CBOR encoding. | chriseth | 2016-12-01 | 5 | -14/+39 |
| | | | | | |||||
| * | | | | Fix emscripten build. | chriseth | 2016-12-01 | 1 | -2/+2 |
| | | | | | |||||
| * | | | | Fix documentation error. | chriseth | 2016-12-01 | 1 | -0/+2 |
| | | | | | |||||
| * | | | | Fix tests. | chriseth | 2016-12-01 | 4 | -23/+3 |
| | | | | | |||||
| * | | | | Add test. | chriseth | 2016-12-01 | 1 | -2/+22 |
| | | | | | |||||
| * | | | | Add swarm hash to the end of the bytecode. | chriseth | 2016-12-01 | 8 | -23/+20 |
| | | | | | |||||
| * | | | | Changelog entry. | chriseth | 2016-12-01 | 1 | -0/+1 |
| | | | | | |||||
| * | | | | Documentation. | chriseth | 2016-12-01 | 1 | -0/+119 |
| | | | | | |||||
| * | | | | Make sure some keys are present. | chriseth | 2016-12-01 | 2 | -1/+4 |
| | | | | | |||||
| * | | | | Incorporate comments. | chriseth | 2016-12-01 | 1 | -10/+10 |
| | | | | | |||||
| * | | | | Metadata stamp. | chriseth | 2016-12-01 | 16 | -54/+171 |
|/ / / / | |||||
* | | | | Merge pull request #1462 from ethereum/fix-incorrect-assertion | chriseth | 2016-12-01 | 2 | -2/+2 |
|\ \ \ \ | | | | | | | | | | | codegen: assertion did not assert non-nullness | ||||
| * | | | | test: more precise expectation about the error for assigning into storage in ↵ | Yoichi Hirai | 2016-12-01 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | inline assembly | ||||
| * | | | | codegen: assertion did not assert non-nullness | Yoichi Hirai | 2016-12-01 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | This commit strengthens an assertion so that it makes sure that a pointer is not null. Moreover, `isLocalVariable(variable)` is now positively asserted, following the error message. | ||||
* | | | | | Merge pull request #1461 from ethereum/reference-not-pointer | chriseth | 2016-12-01 | 1 | -7/+7 |
|\ \ \ \ \ | | | | | | | | | | | | | test: replace a pointer argument by a reference | ||||
| * | | | | | test: change retrieveFunctionBySignature to take a constant reference rather ↵ | Yoichi Hirai | 2016-12-01 | 1 | -7/+7 |
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | than a pointer The function `retrieveFunctionBySignature` expected the argument to be a non-null pointer. This commit makes this assumption more explicit. | ||||
* | | | | | Merge pull request #1460 from ethereum/unused-total | chriseth | 2016-12-01 | 1 | -3/+2 |
|\ \ \ \ \ | | | | | | | | | | | | | libevmasm: remove an unused variable `total` | ||||
| * | | | | | libevmasm: remove an unused variable `total` | Yoichi Hirai | 2016-12-01 | 1 | -3/+2 |
| |/ / / / | |||||
* | | | | | Merge pull request #1463 from ethereum/duplicate-assignment | chriseth | 2016-12-01 | 1 | -2/+0 |
|\ \ \ \ \ | | | | | | | | | | | | | codegen: this commit removes one of duplicate assignments | ||||
| * | | | | | codegen: this commit removes one of duplicate assignments | Yoichi Hirai | 2016-12-01 | 1 | -2/+0 |
| |/ / / / | |||||
* | | | | | Merge pull request #1458 from federicobond/r-literals | chriseth | 2016-12-01 | 2 | -719/+936 |
|\ \ \ \ \ | | | | | | | | | | | | | Use more R string literals in tests | ||||
| * | | | | | Use more R string literals in tests | Federico Bond | 2016-12-01 | 2 | -719/+936 |
| | |_|/ / | |/| | | | |||||
* | | | | | Merge pull request #1455 from ethereum/doc-number-literal-format | Alex Beregszaszi | 2016-12-01 | 1 | -0/+8 |
|\ \ \ \ \ | | | | | | | | | | | | | docs: describe the form of number literals | ||||
| * | | | | | docs: describe the form of number literals | Yoichi Hirai | 2016-12-01 | 1 | -0/+8 |
| |/ / / / | | | | | | | | | | | | | | | | This is a part of #1390 | ||||
* | | | | | Merge pull request #1313 from ethereum/lll-strict-parser | chriseth | 2016-12-01 | 3 | -15/+11 |
|\ \ \ \ \ | | | | | | | | | | | | | LLL: stricter parser | ||||
| * | | | | | LLL: update tests with the strict parser | Alex Beregszaszi | 2016-12-01 | 1 | -1/+3 |
| | | | | | | |||||
| * | | | | | LLL: throw exceptions on invalid symbols | Alex Beregszaszi | 2016-12-01 | 1 | -12/+6 |
| | | | | | | |||||
| * | | | | | LLL: () requires at least one parameter | Alex Beregszaszi | 2016-12-01 | 1 | -1/+1 |
| | | | | | | |||||
| * | | | | | LLL: do not accept '0x' as 0 | Alex Beregszaszi | 2016-12-01 | 1 | -1/+1 |
| |/ / / / | |||||
* | | | | | Merge pull request #1454 from ethereum/end-of-comment | chriseth | 2016-12-01 | 3 | -1/+31 |
|\ \ \ \ \ | |_|/ / / |/| | | | | parser: recognize an end of comment of the form `**/` |