Skip to content

Script header#903

Merged
shargon merged 30 commits intoneo-project:masterfrom
shargon:script-header
Jul 29, 2019
Merged

Script header#903
shargon merged 30 commits intoneo-project:masterfrom
shargon:script-header

Conversation

@shargon
Copy link
Copy Markdown
Member

@shargon shargon commented Jul 9, 2019

Close #889

@shargon shargon requested a review from erikzhang July 9, 2019 16:59
erikzhang
erikzhang requested changes Jul 10, 2019
View reviewed changes
shargon
shargon commented Jul 11, 2019
View reviewed changes
@codecov-io
Copy link
Copy Markdown

codecov-io commented Jul 11, 2019 •
edited
Loading

Codecov Report

Merging #903 into master will increase coverage by 0.12%.
The diff coverage is 86.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #903      +/-   ##
==========================================
+ Coverage   52.05%   52.17%   +0.12%     
==========================================
  Files         178      179       +1     
  Lines       12662    12707      +45     
==========================================
+ Hits         6591     6630      +39     
- Misses       6071     6077       +6
Impacted Files Coverage Δ
neo/SmartContract/NefFile.cs 86.66% <86.66%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a9f46bc...0d7f22e. Read the comment docs.

erikzhang
erikzhang reviewed Jul 14, 2019
View reviewed changes
@shargon shargon requested review from erikzhang and igormcoelho July 14, 2019 13:26
erikzhang
erikzhang reviewed Jul 15, 2019
View reviewed changes
igormcoelho
igormcoelho reviewed Jul 17, 2019
View reviewed changes
igormcoelho
igormcoelho reviewed Jul 17, 2019
View reviewed changes
neo/SmartContract/NefFile.cs Outdated
public void Serialize(BinaryWriter writer)
{
writer.Write((int)Magic);
writer.WriteFixedString(Compiler, 32);
Copy link
Copy Markdown
Contributor

@igormcoelho igormcoelho Jul 17, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

32 bytes compiler? isn't it much?

Copy link
Copy Markdown
Member

@erikzhang erikzhang Jul 22, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more. 

Microsoft Visual Studio 2019

It consumes 28 bytes.

igormcoelho
igormcoelho reviewed Jul 17, 2019
View reviewed changes
Copy link
Copy Markdown
Contributor

@igormcoelho igormcoelho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

must see how to load this, and if these prefixes aren't too much for a binary nvm.

igormcoelho
igormcoelho reviewed Jul 17, 2019
View reviewed changes
igormcoelho
igormcoelho reviewed Jul 17, 2019
View reviewed changes
neo/SmartContract/NefFile.cs Outdated
/// <summary>
/// Version
/// </summary>
public Version Version { get; set; }
Copy link
Copy Markdown
Contributor

@igormcoelho igormcoelho Jul 17, 2019 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which version is this? Neo (core) version? NeoVM version? Perhaps both?
But since every Neo (core) release carries a specific NeoVM version, it's fine to just use Neo (core) version I guess.

Or is it Compiler + Devpack version? hahaha too many versions... perhaps it's compiler related this, right? It would be good to have some minimal protocol version here, to differentiate between Neo2 and Neo3, and perhaps other affiliated Neo protocols.

Copy link
Copy Markdown
Member Author

@shargon shargon Jul 19, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(Mayor, Minor, Build, Version)

shargon and others added 2 commits July 19, 2019 13:05
@shargon
Remove code
d0e2bc6
@erikzhang @igormcoelho
Update neo/SmartContract/NefFile.cs
a75e6f4 
Co-Authored-By: Igor Machado Coelho <igor.machado@gmail.com>
erikzhang
erikzhang reviewed Jul 22, 2019
View reviewed changes
@erikzhang
Copy link
Copy Markdown
Member

erikzhang commented Jul 22, 2019

Also we can add here the description, author, email (if we will request this in the deploy)

Now we don't request these in the deployment.

@erikzhang erikzhang mentioned this pull request Jul 22, 2019
Merged
@shargon
Copy link
Copy Markdown
Member Author

shargon commented Jul 22, 2019

If we want to accept ONT file format, is easier with an enum

@erikzhang
Copy link
Copy Markdown
Member

erikzhang commented Jul 22, 2019

If we want to accept ONT file format, is easier with an enum

For ONT, we have System.Runtime.Platform.

@erikzhang
Copy link
Copy Markdown
Member

erikzhang commented Jul 23, 2019 •
edited
Loading

Should we move it to neo-vm?

@shargon
Copy link
Copy Markdown
Member Author

shargon commented Jul 23, 2019

If is moved to neo-vm we need to remove the hash

erikzhang
erikzhang reviewed Jul 24, 2019
View reviewed changes
@erikzhang erikzhang mentioned this pull request Jul 25, 2019
Closed
@shargon shargon requested a review from erikzhang July 26, 2019 08:21
@shargon
Copy link
Copy Markdown
Member Author

shargon commented Jul 26, 2019

i think that we should add the ABI, are you agree @erikzhang?

@erikzhang
Copy link
Copy Markdown
Member

erikzhang commented Jul 28, 2019

i think that we should add the ABI

I prefer ABI in a separate file.

@vncoelho vncoelho mentioned this pull request Jul 28, 2019
Merged
@shargon shargon merged commit f4adfac into neo-project:master Jul 29, 2019
@shargon shargon deleted the script-header branch July 29, 2019 05:25
@erikzhang erikzhang added this to the NEO 3.0 milestone Jul 29, 2019
@vncoelho
Copy link
Copy Markdown
Member

vncoelho commented Jul 29, 2019

Here we go!

Thacryba pushed a commit to simplitech/neo that referenced this pull request Feb 17, 2020
@Celia18305
add big and little endian usage (neo-project#903)
5ead6fa 
* add big and little endian usage

* Update endian.md

* Update endian.md

* Update endian.md

* Update endian.md

* Update endian.md
@Tommo-L Tommo-L mentioned this pull request Mar 26, 2020
Closed
Tommo-L pushed a commit to Tommo-L/neo that referenced this pull request Jun 22, 2020
@shargon @Tommo-L
Script header (neo-project#903)
c47f2e3 
* Script header

* Script header draft

* Clean files

* Typo

* Rename class and magic header

* Clean hash

* UT

* Erik's suggestions

* Change name

* Typo

* Check magic

* 64 to 32

* Nef file only for neo-cli

* Real Checksum

* Clean

* Clean again

* Remove code

* Update neo/SmartContract/NefFile.cs

Co-Authored-By: Igor Machado Coelho <igor.machado@gmail.com>

* Enum to const

* Remove unnecessary change

* Apply Erik's suggestions
@roman-khimov roman-khimov mentioned this pull request Jul 6, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@erikzhang erikzhang erikzhang approved these changes

@igormcoelho igormcoelho Awaiting requested review from igormcoelho

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

NEO 3.0

Development

Successfully merging this pull request may close these issues.

Script header for multiple virtual machines

5 participants

@shargon @codecov-io @erikzhang @vncoelho @igormcoelho