search

ctm / mb2-doc

Mb2, poker software
https://devctm.com
7 stars 2 forks source link

DRY upcoming::event_row.rs and running::event_row.rs #1451

Closed ctm closed 5 days ago

ctm commented 6 days ago

Hoist common code to events.rs.

Currently, the two event_row.rs files have a ridiculously large amount of duplicated code. In theory, I was going to refactor it every time I needed to touch anything that was in two places, but in reality, I either forgot or was rushed.

So, while I had a block of time too small to do much else, I did a little of the DRYing, mostly as a proof-of-concept (and to not fritter and waste the small block of time in an offhand way).

I'll deploy what I've done now and get on with my day, but I'll keep this issue open until I've picked all the low-hanging fruit.

ctm commented 5 days ago

I got another chunk done yesterday afternoon and almost finished the work before a late walk with mom. I've since finished the work and am deploying now.