On 2021-04-03, Michael Forney <mfor...@mforney.org> wrote: > On 2020-01-28, Michael Forney <mfor...@mforney.org> wrote: >> On 2019-05-28, Michael Forney <mfor...@mforney.org> wrote: >>> I noticed that OpenBSD's fscanf doesn't yet support hex float strings, >>> which are standardized in C99. I am using them in my application (which >>> I would like to support OpenBSD), since the "%a" format specifier is a >>> convenient way to preserve the exact floating point value. >>> >>> strtod already supports parsing hex floats, so it is just the scanner >>> in __svfscanf that needed changes. >>> >>> The implementation reuses the PFXOK and NZDIGITS flags from CT_INT >>> scanning and follows similar logic to CT_INT. This required allocating >>> new flag values for DPTOK and EXPOK. >>> >>> I did my best to follow style(9), but since the indentation level of >>> this >>> switch is so high, I found it difficult wrap lines nicely. I noticed >>> that >>> several existing lines broke the "space around binary operators" rule if >>> the added space would require unnatural wrapping, so I did the same >>> here. >>> >>> I wasn't sure which comments I should carry over from the CT_INT case >>> (for example, above `case 'x':`), or if any of the other changes require >>> additional comments. Please let me know if they do. >> >> Just bumping this so it doesn't get forgotten. I'd be happy to address >> any feedback anyone might have. > > Ping.
Ping again.