clear build warning list

[s1Sharp]

remove warnings like:

• src/engines/KeyValue.cpp: warning: unused parameter 'params' [-Wunused-parameter]
• src/engines/KeyValue.cpp: warning: field 'version' will be initialized after field 'client' [-Wreorder-ctor]

[s1Sharp]

It looks like there are formatting changes along with the relevant updates. Can you please resubmit without the formatting changes to make the actual changes more apparent?

I've made a few changes.

• First, I added [[maybe-unused]] C++ attribute to avoid warnings that I faced
• Second, I only changed the order of initialization of class members, which does not affect the functionality in any way (class members do not depend on each other during initialization). I didn't find strict formatting rules and decided to use the approach: 1 line for 1 class member (more readable in my opinion). If you think that readability has gone, I will return the approach of initializing multiple members on one line.

[s1Sharp]

now member in class and in constructor member init sequence the same

  bool debug_;
  bool tls_;
  Host host_;
  Port port_;
  Token token_;
  Namespace namespace_;
  HttpClient httpClient_;
  AuthenticationStrategy &authStrategy_;
  std::filesystem::path caBundle_;

  : debug_(config.getDebug())
  , tls_(config.getTls())
  , host_(config.getHost())
  , port_(config.getPort())
  , namespace_(config.getNamespace())
  , httpClient_(HttpClient(config))
  , authStrategy_(authStrategy)
  , caBundle_(config.getCaBundle())
  // + `Token token` init in constructor body {...}

[abedra]

Thank you!