|
 |
Am 12.07.2016 um 20:41 schrieb dick balaska:
>> Commit Summary
>>
>> Refactoring IOStream family of classes.
>
> Oh please oh please oh please, let Him be refactoring out that 80 that
> is hardcoded as a linewidth in a boatload of constructors.
Uh... no, that's way beside my focus, sorry to disappoint you. I'm
primarily concerned with file input at the lowest level (actual file
access, implemented by the IOBase(*) family of classes), not file output
at the highest level (word wrapping, implemented by the TextStreamBuffer
family of classes).
(*erroneously referred to in the commits and pull request as IOStream)
Post a reply to this message
|
|
|
|
Am 2016-07-12 17:39, also sprach clipka:
> Am 12.07.2016 um 20:41 schrieb dick balaska:
>>> Commit Summary
>>>
>>> Refactoring IOStream family of classes.
>>
>> Oh please oh please oh please, let Him be refactoring out that 80 that
>> is hardcoded as a linewidth in a boatload of constructors.
>
> Uh... no, that's way beside my focus, sorry to disappoint you. I'm
> primarily concerned with file input at the lowest level (actual file
> access, implemented by the IOBase(*) family of classes), not file output
> at the highest level (word wrapping, implemented by the TextStreamBuffer
> family of classes).
>
> (*erroneously referred to in the commits and pull request as IOStream)
>
Yeah... I know.
I had that thought, then looked at the diff and saw what you were doing,
but decided to post that anyway.
Back when I first brought it up I thought fixing it would be a good toe
dipping exercise, but no. It's not a config parameter, it's a friggin
parameter passed in a constructor! (And it's not even a symbol, like
CONSOLEWIDTH, it's a freakin numeric 80! Argh.)
I still want to fix that, but I have more impovtant things I'm working on...
--
dik
Post a reply to this message
|
|
