Conversation
|
|
Thanks for your contribution. |
damorinCan
changed the title
|
Title now including reference to C11 and removed reference to Windows 11.
I added a test in test/testio.cpp. Anything missing in it ? |
Thanks, I must have missed that. |
|
Ping @danmar |
danmar
left a comment
danmar
left a comment
There was a problem hiding this comment.
I feel we need to have a ticket for this in trac
it feels like it should be added to the release notes as a "new check" ?
| case Filepointer::Operation::UNIMPORTANT: | ||
| if (f.mode == OpenMode::CLOSED) | ||
| useClosedFileError(tok); | ||
| if (isftell && windows && f.read_mode == Filepointer::ReadMode::READ_TEXT && printPortability) |
There was a problem hiding this comment.
the error message says something about the C11 standard. Nothing about windows.
| " (void)ftell(f);\n" | ||
| " fclose(f);\n" | ||
| " }\n" | ||
| "}\n", dinit(CheckOptions, $.platform = Platform::Type::Win32A, $.portability = true)); |
There was a problem hiding this comment.
test what happens on unix platform . test what happens when C99 is used - that works as expected right?
3 participants
Some legacy tools stopped suddenly working after 20+ years.
https://stackoverflow.com/questions/79762122/ftell-no-more-returning-the-correct-offset-on-a-text-file-with-windows-11-ente