Skip to content

Commit

Permalink
Merge pull request #79 from kant/patch-1
Browse files Browse the repository at this point in the history
Minor proposal (paragraph 311)
  • Loading branch information
danomatika committed Mar 19, 2021
2 parents 586a495 + b6a6e36 commit 56f1510
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion README.md
Expand Up @@ -308,7 +308,7 @@ void MyClass::setup() {

### ofxMidi classes created in constructors don't seem to work

This is related to the issue above, in that the ofxMidi classes are being created too early in the app startup process and the back end MIDI library is not being set up correctly. The easiest & best solution is to call the ofxMidi class setup code as part of you ofApp's setup() function, whether there directly or within a subclass. This way you have direct control over when things are happening as opposed to within a constructor which may be called at an arbitrarily early point.
This is related to the issue above, in that the ofxMidi classes are being created too early in the app startup process and the back end MIDI library is not being set up correctly. The easiest & best solution is to call the ofxMidi class setup code as part of your ofApp's setup() function, whether there directly or within a subclass. This way you have direct control over when things are happening as opposed to within a constructor which may be called at an arbitrarily early point.

DEVELOPING
----------
Expand Down

0 comments on commit 56f1510

Please sign in to comment.