On Wed, 31 Mar 2021, Sven Schreiber wrote:
Am 30.03.2021 um 15:37 schrieb Allin Cottrell:
> On Mon, 29 Mar 2021, Sven Schreiber wrote:
>> Syntactically, shouldn't it rather be getline(source, &target) ? I
>> thought that in hansl a function argument can only be modified if a
>> pointer to it is passed. Compare the mols() or the eigen() syntax. Are
>> there other functions with writable non-pointerized arguments?
>>
>> (In any case, the old form could maybe continue to be allowed for
>> compatibility.)
>
> You're right, it would be more consistent to require (at least
> nominally) a pointerized argument for target.
>
> I took a look at geneval.c to see if by any chance the pointerized
> version was already accepted. Answer: No. But it would be quite easy
> to support the pointer version and revise the doc accordingly, while
> keeping the existing form for compatibility.
Sounds good! (Even though it adds another layer of version compatibility
complexity to function packages, but I think it's better in the medium
term already.)
That change in now made and documented in git. I think it's OK if
packages continue to use the prior getline signature, since the
change is really just cosmetic.
Allin