romalpeccia / PolyGnome

PolyGnome: MIDI Sequencer Plugin - create polyrhythmic MIDI sequences in your Digital Audio Workstation in real time
2 stars 0 forks source link

fpermissive compiler flag needed in Linux build #5

Open crshrprt opened 9 months ago

crshrprt commented 9 months ago

Trying to build it on Linux, Debian Sid, I got some errors:

In file included from ../../Source/PolyRhythmMachine.cpp:25:
../../Source/PolyRhythmMachine.h:68:10: error: extra qualification ‘PolyRhythmMachine::’ on member ‘handleNoteTrigger’ [-fpermissive]
   68 |     void PolyRhythmMachine::handleNoteTrigger(juce::MidiBuffer&, int noteNumber, int velocity, int bufferPosition);
      |          ^~~~~~~~~~~~~~~~~
In file included from ../../Source/PluginProcessor.h:27,
                 from ../../Source/PluginEditor.cpp:22:
../../Source/PolyRhythmMachine.h:68:10: error: extra qualification ‘PolyRhythmMachine::’ on member ‘handleNoteTrigger’ [-fpermissive]
   68 |     void PolyRhythmMachine::handleNoteTrigger(juce::MidiBuffer&, int noteNumber, int velocity, int bufferPosition);
      |          ^~~~~~~~~~~~~~~~~
../../Source/Utilities.cpp: In function ‘int midiStringToInt(std::string)’:
../../Source/Utilities.cpp:148:25: warning: NULL used in arithmetic [-Wpointer-arith]
  148 |     if (midiValue[0] != NULL) {
      |                         ^~~~
../../Source/Utilities.cpp:153:25: warning: NULL used in arithmetic [-Wpointer-arith]
  153 |     if (midiValue[1] != NULL) {
      |                         ^~~~
In file included from ../../Source/PluginProcessor.h:27,
                 from ../../Source/PluginProcessor.cpp:24:
../../Source/PolyRhythmMachine.h:68:10: error: extra qualification ‘PolyRhythmMachine::’ on member ‘handleNoteTrigger’ [-fpermissive]
   68 |     void PolyRhythmMachine::handleNoteTrigger(juce::MidiBuffer&, int noteNumber, int velocity, int bufferPosition);
      |          ^~~~~~~~~~~~~~~~~

I need to add -fpermissive compiler flag to be able to build PolyGnome correctly.

2024-02-09_21 59 12 494

romalpeccia commented 9 months ago

Thanks for letting me know! It seems like my compiler wasn't catching these errors. I fixed them in the latest commit.