On Monday, April 23, 2018 at 10:58:08 AM UTC+2, Edward K. Ream wrote:
>
> ... a discussion about programming style ...
>
...
2. The meaning of each kwarg is clear:
- reassignIndices: distinguishes between paste-node and paste-retaining
clones.
- redrawFlag: suppresses calls to c.redraw when False.
- s: The to-be-pasted text. Get the text from the clipboard if None.
- tempOutline: don't make entries in the gnxDict.
- undoFlag: Handle undo unless False.
That made me wonder: Do you plan sometime in the future to make the code
PEP8-compliant? Does Leo have any kind of 'style guide' or naming
conventions?
Just a cursory glance at the parameters shows that at least some
explanations/comments could be removed if other names were used, i.d. if
the names wouldn't express what a paramer *is* (i.e. a 'flag'), but what
*action* it stands for. Examples:
Instead
of
one could use
redrawFlag
redrawOutline
undoFlag enableUndo
tempOutline=False, # True: don't make entries in the gnxDict.
updateGnxDict
--
You received this message because you are subscribed to the Google Groups
"leo-editor" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/leo-editor.
For more options, visit https://groups.google.com/d/optout.
