Poll: Restructure Include Directories ? #1575
Locked
pschatzmann
started this conversation in
Polls
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
The libraries currently provides all the functionality that I have initially planned.
There is however one final topic where I am not sure how to advance.
Currently you can use
#include "AudioTools.h"
to include all the mandatory functionality and you would use e.g.#include "AudioLibs/A2DPStream.h
or#include "AudioCodecs/CodecMP3Helix.h"
to include an optional class.I am proposing too change the optional cases to
#include "AudioTools/AudioLibs/A2DPStream.h"
or#include "AudioTools/AudioCodecs/CodecMP3Helix.h"
to make it better visible that these classes are indeed part of the AudioTools.This would also prevent a code reformatting that re-sorts the includes to make the sketch invalid.
11 votes ·
Beta Was this translation helpful? Give feedback.
All reactions