Fix some more warnings (#1288)

This fixes a number of warnings that were raised with `clang`:

```
./Source/Particles/Gather/ScaleFields.H:44:17: warning: implicit conversion
      turns floating-point number into integer: 'const amrex::Real'
      (aka 'const double') to 'bool' [-Wfloat-conversion]
        if (not m_do_scale) return;
```
I changed the type of the flag `m_do_scale` from an `amrex::Real` to a `bool`

```
./Source/Particles/WarpXParticleContainer.H:335:87: warning: extra ';' after
      member function definition [-Wextra-semi]
  ...resample (const Resampling& /*resampler*/, const int /*timestep*/) {};
```
I removed the corresponding semi-colon.

```
      delete called on non-final 'LabFrameDiag' that has virtual functions but
      non-virtual destructor [-Wdelete-non-abstract-non-virtual-dtor]
    delete __ptr;
```
I added a declaration for a virtual destructor.

```
./Source/Utils/NCIGodfreyTables.H:22:5: warning: suggest braces around
      initialization of subobject [-Wmissing-braces]
```
I added additional curly braces, to separate each line of the declared 2D arrays. (The values inside the array are unchanged.)

Co-authored-by: Axel Huebl <axel.huebl@plasma.ninja>

0 files changed, 0 insertions, 0 deletions