Stream: git-wasmtime

Topic: wasmtime / PR #2089 machinst x64: use assert_eq! when pos...


view this post on Zulip Wasmtime GitHub notifications bot (Aug 03 2020 at 23:21):

abrown opened PR #2089 from refactor-assert to main:

This change is a pure refactoring--no change to functionality. It replaces various forms of assertions (e.g. ==, !=) with their macro form (e.g. assert_eq, assert_ne). I don't truly have a preference either way but it makes the IDE I use stop telling me that there is a "possible code improvement."

<!--

Please ensure that the following steps are all taken care of before submitting
the PR.

Please ensure all communication adheres to the code of conduct.
-->

view this post on Zulip Wasmtime GitHub notifications bot (Aug 03 2020 at 23:49):

cfallin submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 04 2020 at 16:18):

abrown merged PR #2089.


Last updated: Dec 23 2024 at 13:07 UTC