On Thu, 1 Dec 2022 at 18:27, John Naylor <john.nay...@enterprisedb.com> wrote: > I don't see why the non-decimal literal patch needs to be "immediately" > faster? If doing this first leads to less code churn, that's another > consideration, but you haven't made that argument.
My view is that Peter wants to keep the code he's adding for the hex, octal and binary parsing as similar to the existing code as possible. I very much understand Peter's point of view on that. Consistency is good. However, if we commit the hex literals patch first, people might ask "why don't we use bit-wise operators to make the power-of-2 bases faster?", which seems like a very legitimate question. I asked it, anyway... On the other hand, if Peter adds the bit-wise operators then the problem of code inconsistency remains. As an alternative to those 2 options, I'm proposing we commit this first then the above dilemma disappears completely. If this was going to cause huge conflicts with Peter's patch then I might think differently. I feel like it's a fairly trivial task to rebase. If the consensus is that we should fix this afterwards, then I'm happy to delay. David