Skip to content

Update newline character instructions in INSTRUCTIONS.md - #3

Open
bdolgov wants to merge 1 commit into
Daemon125:masterfrom
bdolgov:patch-1
Open

bdolgov wants to merge 1 commit into
Daemon125:masterfrom
bdolgov:patch-1

Conversation

@bdolgov

@bdolgov bdolgov commented Sep 28, 2026

Copy link
Copy Markdown

Clarified instructions regarding newline character handling.

If \n was typed as /n, the number of characters will be different.

What this changes

INSTRUCTIONS.md

Category

  • Documentation correction (factual error, with a source)
  • Portability fix for a platform listed as expected in README section 10.1
  • Typo
  • Pipeline change
  • Something else (see README section 15.2 first)

Verification

I am not running arbitrary code on my devices.

If this changes hello.c

N/A

If this changes a documented figure

N/A

Notes

Clarified instructions regarding newline character handling.

If `\n` was typed as `/n`, the number of characters will be different.
@bdolgov
bdolgov requested a review from Daemon125 as a code owner September 28, 2026 07:36
@Favkes

Favkes commented Sep 28, 2026

Copy link
Copy Markdown

looks legit 👍

@xRaTcHeT302

xRaTcHeT302 commented Oct 1, 2026 •

Copy link
Copy Markdown

i dunno, i feel like we COULD optimize a byte, this seems like a downgrade

have u done regression testing??? WHERE'S THE TEST????

Gross

i'm sorry but until all 172 validation tests pass, this wont do
can we SIMD this???

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants