GNU bug report logs -
#51451
image-transform-original has a misleading name
Previous Next
Reported by: Stefan Kangas <stefan <at> marxist.se>
Date: Thu, 28 Oct 2021 00:32:02 UTC
Severity: minor
Fixed in version 29.1
Done: Stefan Kangas <stefankangas <at> gmail.com>
Bug is archived. No further changes may be made.
Full log
View this message in rfc822 format
close 51451 29.1
thanks
Juri Linkov <juri <at> linkov.net> writes:
>>> image-transform-* commands are modifying the values of
>>> image-transform-* variables.
>>> Would you propose to rename variables as well?
>>
>> I didn't think about that, as the commands are what the user will see.
>> Maybe we should. Do you think it would be a good idea?
>
> At least, currently there is consistency in naming of
> image-transform-* functions and variables. Whether
> selecting such a prefix was a good choice is another question.
>
> But if you see a problem only in the name of image-transform-original,
> then it could be renamed to image-transform-set-original or maybe
> better to image-transform-reset-to-original, i.e. to something more
> descriptive, while keeping the existing name prefix.
>
> PS: renaming to image-transform-reset-to-original will also require
> renaming image-transform-reset to image-transform-reset-to-initial.
Thanks. I've now changed this as you suggest (commit 37fe0cd358).
This bug report was last modified 2 years and 309 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.