Refactor pgp_key_search_t

[ni4]

Another one on the C++-transition way.Thought it would be faster...

[codecov[bot]]

Codecov Report

Attention: 16 lines in your changes are missing coverage. Please review.

Comparison is base (9fd1893) 77.26% compared to head (e40c4ff) 77.29%.

Files	Patch %	Lines
src/lib/key-provider.cpp	89.41%	9 Missing :warning:
src/lib/rnp.cpp	93.67%	5 Missing :warning:
src/lib/types.h	88.88%	1 Missing :warning:
src/librepgp/stream-parse.cpp	88.88%	1 Missing :warning:

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2187      +/-   ##
==========================================
+ Coverage   77.26%   77.29%   +0.02%     
==========================================
  Files         194      194              
  Lines       37751    37692      -59     
==========================================
- Hits        29170    29135      -35     
+ Misses       8581     8557      -24     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[antonsviridenko]

Thought it would be faster...

Is it actually faster now? :P

[antonsviridenko]

There is some conflict, probably caused by the recent merged PR...

[ni4]

Thought it would be faster...

Is it actually faster now? :P

I mean faster transition to C++ from all the anonymous unions, char * and suchlike internal stuff :) I'll rebase and fix errors and ping you back for review, thanks!

[ni4]

@ronaldtse @antonsviridenko ping for review. Spend a lot of time on debugging on some test failure but separated it to the issue #2192 as too much time was spent on it.