Did you close because no longer relevant?

I've started to read the documentation and it looks like it's definitely worth plugging this math emulation layer into libcmb to see just how much performance gain is actually realized.

After thought, I support this change.

@rgrimes @cem If one of you approve, I'll go ahead and usher it in

I approve of this change. But since I have not committed in a while, want to get sign-off from at least one other person.

Try to get ahold of mw@ and loos@ to review

Thanks! I have a huge update coming this month that will bring in lots of changes. Hoping you could revisit just afterward.

Add support for `-i random' for seeking to a random position.
Updated manual with new examples.

Add support for negative `-i num' values
(for seeking to the back-end of a series)

+ cmb(1,3): Fix broken handling of -i num when given -k -1
+ cmb(3): Remove a stray newline
+ cmb(3): Add show_empty option for including the empty-set
+ cmb(1): Add -e option for showing the empty-set

Bugs already fixed for next update:
+ -i NUM was broken with -k -1

Update to address feedback from adridg

In D16132#380314, @adridg wrote:

Comments in general not tied to a specific line of code:

• The -c and -i flags ask for a deterministic order of the combinations. If I use -c 4 -i 1 I expect 4 combinations (whatever they may be). If I use -c 3 -i 2 I expect the last three of those four combinations. So the ordering must be consistent. It is, in your implementation, but the documentation doesn't guarantee that.

cmb(1): Add missing -n option to usage and give a more informative usage
cmb(3): Fix style issue of single-line for-loops
cmb(3): API changes to cmb_print(1) and struct cmb_config

Address feedback and minor improvements:
+ Move static variables out of header (adridg)
+ Remove "All rights reserved" from copyrights (0mp)
+ Add EXAMPLES section to cmb(1) manual (0mp)
+ Add missing $FreeBSD$ (0mp)
+ Update cmb(1,3) HISTORY section [s/12.0/13.0/]
+ Improved error checking for 64-bit integer overflow
+ Fix copy-paste error in cmb(3) manual (/#endif/d)
+ Use errno(2) instead of errx(3) in cmb(3) library
+ Sync with upstream version 0.7 of [lib]cmb
+ Fix style(9) issue of single-line if-statements
+ Add missing $FrauBSD$ keyword to cmb(1) manual
+ Remove redundant RCS keywords from cmb(1)

In D16132#379579, @0mp wrote:

Also, is there a pattern for the use of Qq, Dq and Ql (instead of Dq Li)?

If no additional comments, I'll move ahead without adding the date (which I could not figure out how to add)

Remove WARNS=6 (this is the default)

Make integer overflow with -t print -1 under MK_OPENSSL=no

Update patch for 13.0, bug fix, and libname edits

In D17739#379451, @dteske wrote:

In D17739#379438, @imp wrote:

In D17739#379433, @dteske wrote:

In D17739#379046, @jmg wrote:

oh, don't forget to bump date!

There's no date to bump?

Add one :)

This manual is not written in standard mdoc from what I can see. It's like it was written by martians.
It doesn't have any of the normal anchors I am familiar with.
I'll just shoehorn a date in here somewhere and hope it works. If it does, I'll hoist it up for review.

In D17739#379438, @imp wrote:

In D17739#379433, @dteske wrote:

In D17739#379046, @jmg wrote:

oh, don't forget to bump date!

There's no date to bump?

Add one :)

In D17739#379439, @imp wrote:

Looks good to me...
Not sure about the \f syntax, but other places use it...

In D17739#379046, @jmg wrote:

oh, don't forget to bump date!

Do you have a commit bit or do you need me to ferry this in?

Looks good.

Rebase following r339413 and add patch for rc.conf(5) manual

Fix wrong URL in pkg-descr and update to 0.4.1 (and convert to pkg-plist)

Address feedback from mat@ and update to 0.3.1

Thanks @lme for checking for potential conflict.

Update to 0.1.1

Make `portlint -A' happy

I just thought of something ...

Provide a new offering

Take

This patch is a good effort. Let's keep building upon it and get it ready for when head is thawed on Sep 23. Thank you for resurrecting this topic.

The patch is showing usr.sbin/certctl/trustctl.sh -- is this a phab issue or did the file not get renamed in the patch?

Template suggestion because file is pretty large

Hope this helps

Corrections