Re:Making a Go program 70% faster with a one character change
by ok_dad on Monday November 14, 2022 @03:49PM
(#33600811)
I really hate when people use "errors" for signals. Or, alternately, use the signals mechanisms for actual errors. An error should be "something went wrong", and reading a file to the EOF is not "going wrong", that's just what the "read()" should do if you tell it so!
I like Go's multiple returns and error checking by default, but it definitely should have been implemented with some sort of "Result/Error" type union instead. Go made so many good decisions, that I am frankly amazed they made this one really bad one.
Re:Making a Go program 70% faster with a one character change
by robotresearcher on Monday November 14, 2022 @04:24PM
(#33601336)
If you take the position that the semantics of read() are 'read data from this endless stream', then EOF is an error indicating that this model no longer applies.
Doesn't bother me at all.
Re:Making a Go program 70% faster with a one character change
by account42 on Tuesday November 15, 2022 @04:42AM
(#33607156)
Except in that model it would be fine to return an error as soon as soon as you know there is an EOF while in the real world you do want to read right up to the EOF because EOF is usually not an error condition but expected, even with streams of indeterminate length.
Re:Making a Go program 70% faster with a one character change
by robotresearcher on Tuesday November 15, 2022 @11:30AM
(#33611296)
The model is that you don’t know there’s an EOF until you’ve read the last byte.
Re:Making a Go program 70% faster with a one character change
by bouncycastle on Wednesday November 16, 2022 @07:36AM
(#33622059)
But that's what "errors" are, the're signals.
Re:Making a Go program 70% faster with a one character change
by nemo1618 on Monday November 14, 2022 @03:53PM
(#33600852)
Presumably this is for performance reasons: if a `Read` call involves, say, a network request, then returning "800, EOF" in one roundtrip is certainly nicer than "800, nil" -> "0, EOF" in two roundtrips. In practice... I suspect this happens rarely enough that it's not worth the cost. The performance edge case could be handled with an alternative API, e.g. an optional EOF() method.
If you try to read 1000 bytes, you might get “800 bytes read, EOF” as a result. The worst part is that os.File won’t ever do this!