Code review comment for lp:~fault/wingpanel-indicator-network/fix-1628541

Revision history for this message
Casper Christiansen (fault) wrote :

> Maybe you can get the priority by creating a `int get_priority ()` in the
> Network.State enum so you don't have to reiterate every time but only do a
> `score = inter.state.get_priority ();`

You're right, that would be better performance wise. I just realized I even forgot to break the loop early once the score was found. Woops.

« Back to merge proposal