| 21:31 | <Michael Ficarra> | I'm running a few minutes late |
| 22:30 | <shu> | ljharb: oops, what'd you say? |
| 22:30 | <ljharb> | just asking about the merge label on 3409 and 3410, but michael's doing it |
| 22:33 | <Michael Ficarra> | @ljharb so next PR is refactoring that lookup table and removing the Type AO? |
| 22:33 | <ljharb> | yep. first i have to come up with an idea to refactor the lookup table tho :-) |
| 22:33 | <ljharb> | if not in the next week, i've got 2 cross-country flights the following week so i'll have plenty of thinkin time |
| 23:04 | <Michael Ficarra> | I think just use 2 tables and do the type discrimination in the algorithm steps before referring to which table to look at |
| 23:09 | <ljharb> | i literally was just looking at it now, and i agree |
| 23:09 | <ljharb> | (it's already weird that bigint has 4 of its ops outside the table, but other ops in the table) |
| 23:10 | <ljharb> | i'll make the PR today or tomorrow, that splits it into a number table and a bigint table. |
| 23:14 | <Michael Ficarra> | 👍️ thanks |