ExtremeFiretop / MerlinAutoUpdate-Router

Merlin(A)uto(U)pdate is a Merlin router script which allows you to remotely identify a stable firmware update for an ASUS Merlin router, and automatically download and update via an unattended method directly from the router.
https://www.snbforums.com/threads/merlinau-v1-2-7-the-ultimate-firmware-auto-updater-amtm-addon.91326/
GNU General Public License v3.0
21 stars 1 forks source link

Code improvement #210

Closed Martinski4GitHub closed 6 months ago

Martinski4GitHub commented 6 months ago

Minor code improvement.

ExtremeFiretop commented 6 months ago

I like this.

The functionality is the same, but this offers a more streamlined and cleaner approach to handling the incrementation logic and the construction of the dow_expr variable when the days wrap around the week. This makes it easier to understand even though the core functionality and performance implications between the two are likely the same.

This brings me back to our last discussion about simplicity for the day me and you are no longer working on this, whatever we can do to make it easier to read I'm all for it!