On Mon, 16 Dec 2024 11:53:58 +0100 Petr Machata wrote:
> > + if ifindex not in parsed:
> > + parsed[ifindex] = {"rx":[], "tx": []}
> > + parsed[ifindex][entry["queue-type"]].append(entry['queue-id'])
>
> BTW setdefault() exists for exactly these add-unless-already-exists
> scenarios:
>
> parsed_entry = parsed.setdefault(ifindex, {"rx":[], "tx": []})
> parsed_entry[entry["queue-type"]].append(entry['queue-id'])
>
> Sometimes this can be used to inline the whole expression, such as
> mydict.setdefault(key, []).append(value), but that would be unwieldy here.
Ack, I used setdefault() initially but it made the line too
incomprehensible. Too many things get indexed at once..
