Skip to content
This repository was archived by the owner on Jun 24, 2026. It is now read-only.

Fix TypeGuard Errors in WASM Module#1601

Merged
bradlarsen merged 6 commits into
masterfrom
dev-typeguard
Apr 9, 2020
Merged

Fix TypeGuard Errors in WASM Module#1601
bradlarsen merged 6 commits into
masterfrom
dev-typeguard

Conversation

@ehennenfent

Copy link
Copy Markdown
Contributor

Updates several type hints so that typeguard will pass

@ehennenfent
ehennenfent requested a review from bradlarsen April 8, 2020 22:24

@bradlarsen bradlarsen left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I actually just made many of these same changes on my typeguard branch (#1655)...

Looks good to me!

@bradlarsen
bradlarsen merged commit d082284 into master Apr 9, 2020
@bradlarsen
bradlarsen deleted the dev-typeguard branch April 9, 2020 21:13
ekilmer added a commit that referenced this pull request Apr 23, 2020
* master: (28 commits)
  Rework syscall invocation for proper behavior under typeguard (#1672)
  printable_bytes (#1671)
  Fix several incorrect type hints (#1668)
  Add type hints to several parts of Manticore (#1667)
  Fix type confusion in manual CPU tests; delete dead testing code (#1669)
  Rework WASM Imports and Fix Typos (#1666)
  Work on "Model is not Available" errors in tests (#1659)
  Fix TypeGuard Errors in WASM Module (#1601)
  Remove unused arg ans separate output from workspace (#1651)
  Add a badge to README.md for LGTM (#1647)
  Fix 2 problems in Linux `sys_open` support & add type hints (#1657)
  Fix LGTM Errors (#1656)
  Delay WASM Branch Condition Concretization (#1641)
  Add feature for SymbolicSocket through sys_accept (#1618)
  Add a bunch of type hints & fix a few issues with Linux platform emulation (#1645)
  Bump to CheckoutV2 (#1654)
  Add support for `sys_arm_fadvise64_64` (#1648)
  Add __slots__ to expressions (#1635)
  Swap remaining uses of `Z3Solver()` to use the singleton interface (#1649)
  CI: have pytest report 100 slowest tests (#1646)
  ...
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