Mistuke · GitHub

@mmhat

Copy link Copy Markdown

Contributor

@mmhat

@tomjaguarpaw

Copy link Copy Markdown

Member

This looks pretty plausible to me, but I'm not an expert with the low level details. There are various people who are better qualified. Perhaps one of them can chime in.

@bgamari @hasufell @snoyberg @Mistuke (or someone else?)

@hasufell

Copy link Copy Markdown

Member

There are two differences:

  1. withFilePath uses getFileSystemEncoding, withCString uses getForeignEncoding
  2. withFilePath checks for interior nulls now (not sure since which base version) and barfs out in case it finds any

Tbh, all of those s*ck. But that's another discussion: #252

Mistuke

@mmhat

Copy link Copy Markdown

Contributor Author

  1. withFilePath checks for interior nulls now (not sure since which base version) and barfs out in case it finds any

I think it is since base-4.18.0.0 (GHC 9.6.1).

If there are no further objections: Who is willing to merge this?

@tomjaguarpaw

@tomjaguarpaw

Copy link Copy Markdown

Member

@mmhat

mmhat deleted the 319-null-byte-arguments branch

August 16, 2024 14:14

Read the original on github.com ↗