Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

cleos remove wasm to wast conversion 📦 #10816

Merged
merged 3 commits into from
Oct 19, 2021

Conversation

heifner
Copy link
Contributor

@heifner heifner commented Oct 15, 2021

Change Description

  • Update cleos to not perform wasm to wast conversion.
  • Users can use standard WebAssembly tooling to convert from wasm to wast if needed.
  • Removal announcement: EOSIO Deprecations #7597

Change Type

Select ONE:

  • Documentation
  • Stability bug fix
  • Other
  • Other - special case

Testing Changes

Select ANY that apply:

  • New Tests
  • Existing Tests
  • Test Framework
  • CI System
  • Other

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

@heifner heifner requested a review from spoonincode October 15, 2021 21:15
@@ -6,6 +6,6 @@ namespace eosio { namespace chain {

std::vector<uint8_t> wast_to_wasm( const std::string& wast );
std::string wasm_to_wast( const std::vector<uint8_t>& wasm, bool strip_names );
std::string wasm_to_wast( const uint8_t* data, uint64_t size, bool strip_names );
std::string wasm_to_wast( const uint8_t* data, uint64_t size, bool strip_names, bool check_limits = true );
Copy link
Contributor

Choose a reason for hiding this comment

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

can all the wasm_to_wast stuff be ripped out?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. It is not used, so might as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed.

Copy link
Contributor

Choose a reason for hiding this comment

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

it's probably possible to purge more, down in the libraries/wasm-jit/Source/WAST (and include dir). That would be highly desirable, but maybe not critical at the moment

Copy link
Contributor

Choose a reason for hiding this comment

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

ah, hm, can't remove all too much in there since unit tests since use wast

@heifner heifner changed the title cleos wasm to wast conversion limits 📦 cleos remove wasm to wast conversion 📦 Oct 16, 2021
@heifner heifner merged commit 32cfb91 into develop-boxed Oct 19, 2021
@heifner heifner deleted the EPE-1531-cleos-get-code-dev-boxed branch October 19, 2021 14:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants