Elliot.
Elliot.
@thejohnfreeman @seelabs @scottschurr - looks like this is ready for (re)review. If you won't be able to have an initial look at this within 2 weeks (and provide some feedback),...
@nbougalis if this PR is ready for review and merging (in your opinion), then please avoid force-pushing to the branch. Best practices would require that any changes to the PR...
@nbougalis Instead of rebasing, just do a merge commit (`git merge`). The merge commit resolves merge conflicts with upstream. The merge commit will go away when the PR is merged...
@thejohnfreeman what's the status of your review of this?
@seelabs will you be able to review this?
@nbougalis - does this PR still make sense to merge? If so, please consider the above comments and update the branch to be up-to-date with `develop`. If not, then share...
@ckeshava do you still think this change is worth making?
note: this simplifies parts of the NegativeUNL calculation and may add value to the code base
@visualbasic6 can you share your rippled.cfg? (Redact any sensitive information.)
For posterity, some notes re: 6c8d3abfb1d27dab7a03556ea943c5084e892311 > I've tested on two versions of macOS: 10.14 and 13.2. > The patch replaces the use of `__udivti3` within the `Number` multiplication operator....