New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove JuceHeader.h and qualify with juce:: namespace #90

Merged

drowaudio merged 2 commits into Tracktion:develop from sudara:remove_juce_header

Aug 30, 2022

Contributor

sudara commented Aug 27, 2022 •

edited

Loading

green build: https://github.com/sudara/pluginval/actions/runs/2940195250

Does what it says on the box:

Removes JuceHeader.h and replaces with minimum necessary JUCE headers.
Qualifies everything juce with juce:: namespace.
Fixed one or two typos
Adds some Clion IDE stuff to .gitignore

That's it! Took 1 hr. Fun fact: my v key actually fell off my mechanical keyboard from all the cmd-v-ing :)

sudara added 2 commits

August 27, 2022 20:07


          Remove JuceHeader.h and qualify with juce::

a825825


          Fix issue with juce::String spacing

e2ca35a

sudara mentioned this pull request

Fix Linux issue with DocumentWindow #91

Merged

drowaudio merged commit ffe8382 into Tracktion:develop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet