Resolve errors where address is used in if-statement

The path resolves the following errors:

    error: address of 'this->triggerResults_' will always evaluate to
    'true' [-Werror,-Wpointer-bool-conversion]

Taking an address of `triggerResults_` and implicitly converting to bool
will always yield true.

I believe author wanted `triggerResults_.isValid()`.

Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
Change HLTTauDQMOfflineObjects from a class to a struct

`HLTTauDQMOfflineObjects` is declared as a struct in
`DQMOffline/Trigger/interface/HLTTauDQMPath.h`.

Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
Resolve all -Wunused-private-field warnings

The patch resolves all -Wunused-private-field generated warnings:

    private field '???' is not used [-Wunused-private-field]

Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
CutMasks and BinDat are structs, not classes

The patch corrects struct forwarding. `CutMasks` and `BinData` are
structs, not classes.

Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
Fix typo in header guard name

Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
Resolve unwanted implicit conversion

Based on `DQMStore` Doxygen documentation, `Nmin_` and `Nmax_` must be
double. Otherwise the numbers are changed by implicit `double` to `int`
conversion.

    JetMETHLTOfflineSource.cc:1593:27: warning: implicit conversion from
    'double' to 'int' changes value from 0.5 to 0 [-Wliteral-conversion]
    JetMETHLTOfflineSource.cc:1594:26: warning: implicit conversion from
    'double' to 'int' changes value from 9.5 to 9 [-Wliteral-conversion]

Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
Remove extra unneeded parentheses

Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
Merge pull request #6377 from davidlt/resolve-clang-err-warn-DQMOffline-Trigger

Misc errors and warnings cleanup in DQMOffline/Trigger