Do not copy values into local variables

Address last of kib's comments.

In D19902#438246, @jilles wrote:

In D19902#437991, @hrs wrote:

In D19902#437962, @jilles wrote:

Perhaps it is cleaner to pass the int age to be searched for by reference. However, the asymmetry in the compare function is intentional as this allows finding an object with more data given the key, while having only one storage array.

Asymmetry of the two arguments gives more capability of data lookup? I do not see a big difference between passing an int value 22 by casting it to (void *) as in the original patch and passing a pointer to a struct person which contains age = 22, in terms of the lookup operation. Can you elaborate a case in which the symmetry arguments do not work?

The problem is not that it does not work, but that it creates a bogus value of type struct person which only contains an age. Although C's type system is unsound, I still prefer to avoid subverting it like this.

• Don't initialize locals in declaration
• Remove spaces after casts
• Remove excesive blank lines
• Revert to "asymmetric" compare function
• Add blank line before multi-line comment
• Remove return in main()

Addressing most of the issues with the revision:

In D19902#437776, @bcr wrote:

Reopen this based on comments by Konstantin and Bruce after the MFC to stable/12. @fernape: Can you look into their comments and address them in a followup patch? Thank you!

Is there anything else that needs to be done?

Properly escape \n sequences

• Eliminate cast to different size warning
• Replace comments with assertions

Fix typo

In D19481#417839, @tcberner wrote:

Yea, looks good to me.

Thanks!

If there are no other objections, I'll go ahead and commit this one.

• Remove CMAKE_ARGS since they are the default
• Remove LDFLAGS and modify src/CMakeLists.txt instead
• Update pkg-plist to match actual installation paths

Add actual port to review.