On 9/25/20 7:55 AM, Markus Armbruster wrote:
John Snow <js...@redhat.com> writes:
On 9/23/20 10:55 AM, Cleber Rosa wrote:
Do you have a use case for returning the level? If not, I'd go
without it, and add a "level" property instead, as it'd serve more
cases.
__int__ is doing that lifting. I can remove the return.
I like my functions to return something useful.
Use your judgement.
Eh, Cleber had a point. Nothing uses it.
(AKA: I already made the edit ...)
Like you say, we'll figure out the truly beautiful way to do code
generation when we tackle this all together, holistically. What I've got
works for now (and isn't terribly complex), let's just roll with it
while we fry the bigger fish.
--js