Обзор дефектов кода музыкального софта. Часть 3. Rosegarden

V670 редкая, скорее всего, потому что gcc (да и вообще многие компиляторы) ругаются на разный порядок переменных в объявлении и в конструкторе.

Скажем, вот на такое

#include <iostream>
using namespace std;

struct Test {
    int var1;
    int var2;
    int var3;
    Test() : var2(42), var1(var2), var3(var1) {}

    void run() {
        cout << var1 << " " << var2 << " " << var3 << endl;
    }
};

int main() {
    Test test;
    test.run();
    return 0;
}

gcc ругается как-то так

test.cpp: In constructor 'Test::Test()':
test.cpp:6:6: warning: 'Test::var2' will be initialized after [-Wreorder]
  int var2;
      ^~~~
test.cpp:5:6: warning:   'int Test::var1' [-Wreorder]
  int var1;
      ^~~~
test.cpp:8:2: warning:   when initialized here [-Wreorder]
  Test() : var2(42), var1(var2), var3(var1) {}
  ^~~~

а clang чуть поумнее и поэтому ругается как-то так:

test.cpp:8:11: warning: field 'var2' will be initialized after field 'var1'
      [-Wreorder]
        Test() : var2(42), var1(var2), var3(var1) {}
                 ^
test.cpp:8:26: warning: field 'var2' is uninitialized when used here [-Wuninitialized]
        Test() : var2(42), var1(var2), var3(var1) {}
                                ^
2 warnings generated.

bool
MidiFile::write(const QString &filename)
{
  std::ofstream *midiFile =
    new std::ofstream(filename.toLocal8Bit(),
                        std::ios::out | std::ios::binary);

  if (!(*midiFile)) {
    RG_WARNING << "write() - can't write file";
    m_format = MIDI_FILE_NOT_LOADED;
    return false;
  }
  ....
  midiFile->close();

  return true;
}