GitLab

This will fix build failure due to the mixture of these in ?:.  (and in any
event when we can make something const that's generally good to do)

This reverts commit 662233a1, reversing
changes made to b41b7dc3.

This reverts commit 8dcf5eeb.

for findOnNameResult.
also removed obsolete note from the findNoNameResult description.

as const.

for the positive case to a separate method and used it directly from find()
and findWildcardMatch().  Moved detailed comments of findWildcardMatch to
the head of its definition so that the function body will be physically
concise.

also be more strict of 'target': we don't (yet) support non NULL case for it
in this Python version anyway, so it must be None.  Added a check for it.
docstring was also updated accordingly.

Noticed while examining the file.

The only reason this wasn't picked up before was because this header
file was never included more than once in a .cc file.

Just a naming convention