Skip to content

Change if...else to non-zero on control_flow page#38992

Merged
Josh-Cena merged 1 commit intomdn:mainfrom
chris-c-mcintyre:main
Apr 6, 2025
Merged

Change if...else to non-zero on control_flow page#38992
Josh-Cena merged 1 commit intomdn:mainfrom
chris-c-mcintyre:main

Conversation

@chris-c-mcintyre
Copy link
Copy Markdown
Contributor

Description

Update if...else conditional in WebAssembly instruction references control_flow page, from "1" to "non-zero".

Motivation

Current information regarding if...else behaviour located on the control_flow reference page is incorrect and misleading, having been (understandably) overlooked when this information was clarified elsewhere on the site.

Additional details

Related issues and pull requests

Relates to #33959
Relates to #20193

(My apologies for any excessive spamming of the above references)

@chris-c-mcintyre chris-c-mcintyre requested a review from a team as a code owner April 6, 2025 01:28
@chris-c-mcintyre chris-c-mcintyre requested review from hamishwillee and removed request for a team April 6, 2025 01:28
@github-actions github-actions bot added Content:wasm WebAssembly docs size/xs [PR only] 0-5 LoC changed labels Apr 6, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2025

Copy link
Copy Markdown
Member

@Josh-Cena Josh-Cena left a comment

Choose a reason for hiding this comment

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

Nice catch, thanks

@Josh-Cena Josh-Cena merged commit 46853a1 into mdn:main Apr 6, 2025
7 checks passed
wbamberg added a commit to wbamberg/content that referenced this pull request Apr 6, 2025
* upstream/main: (158 commits)
  fix: remove unrelated add() method reference from openCursor() doc (mdn#38998)
  Fix link (mdn#38995)
  Remove uncovered topic from a tuturial overview. (mdn#38999)
  Change the description of Temporal.Instant (mdn#38996)
  Remove `port` mention for URL `protocol` documentation (mdn#39000)
  Improve the clarity of a sentence (mdn#39003)
  Change if...else to non-zero on control_flow page (mdn#38992)
  Update group-data of EditContext API (mdn#38842)
  improve description (mdn#38845)
  fix: auto-cleanup by bot (mdn#38990)
  New pages: HTMLFormElement.rel and .relList (mdn#38900)
  Add live sample embeds for SVG path examples (mdn#38928)
  Remove uncovered topic from the introduction (mdn#38986)
  Remove self-links (mdn#38982)
  Synchronize with BCD v6.0.3 (mdn#38983)
  Fix grammar (mdn#38981)
  Remove external Web Speech API articles (mdn#38980)
  Add missing {{Specifications}} macros (mdn#38975)
  Remove browser-compat for CSS types without BCD (mdn#38970)
  Update async_function_star_ to have a working example (mdn#38972)
  ...
cssinate pushed a commit to cssinate/content that referenced this pull request Apr 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:wasm WebAssembly docs size/xs [PR only] 0-5 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants