13
0
livetrax/libs/midi++2
David Robillard e0aaed6d65 *** NEW CODING POLICY ***
All #include statements that include a header that is a part of a library
bundled with ardour MUST use quotes, not angle brackets.

Do this:

#include "ardour/types.h"

NOT this:

#include <ardour/types.h>

Rationale:

This is best practice in general, to ensure we include the local version
and not the system version.  That quotes mean "local" (in some sense)
and angle brackets mean "system" (in some sense) is a ubiquitous
convention and IIRC right in the C spec somewhere.

More pragmatically, this is required by (my) waf (stuff) for dependencies
to work correctly.  That is:

!!! FAILURE TO DO THIS CAN RESULT IN BROKEN BUILDS !!!

Failure to comply is punishable by death by torture. :)

P.S. It's not that dramatic in all cases, but this (in combination with some
GCC flags specific to the include type) is the best way I have found to be
absolutely 100% positive the local ones are being used (and we definitely
want to be absolutely 100% positive on that one).


git-svn-id: svn://localhost/ardour2/branches/3.0@4655 d708f5d6-7413-0410-9779-e7cbd77b26cf
2009-02-25 18:26:51 +00:00
..
macosx rollback to 3428, before the mysterious removal of libs/* at 3431/3432 2008-06-02 21:41:35 +00:00
midi++ *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
.cvsignore rollback to 3428, before the mysterious removal of libs/* at 3431/3432 2008-06-02 21:41:35 +00:00
alsa_sequencer_midiport.cc *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
COPYING rollback to 3428, before the mysterious removal of libs/* at 3431/3432 2008-06-02 21:41:35 +00:00
coremidi_midiport.cc *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
fd_midiport.cc *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
fifomidi.cc *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
jack_midiport.cc *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
libmidi++.pc.in rollback to 3428, before the mysterious removal of libs/* at 3431/3432 2008-06-02 21:41:35 +00:00
libmidi++.spec.in rollback to 3428, before the mysterious removal of libs/* at 3431/3432 2008-06-02 21:41:35 +00:00
midi.cc *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
midichannel.cc *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
midifactory.cc *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
midimanager.cc *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
midiparser.cc *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
midiport.cc *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
miditrace.cc *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
midnam_patch.cc Make sure Evoral::MIDIEvent XML stuff is entirely unused in midi++ for now (resolve ticket #0002491). 2009-02-18 18:04:54 +00:00
mmc.cc *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
mmctest.cc *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
mtc.cc *** NEW CODING POLICY *** 2009-02-25 18:26:51 +00:00
SConscript Factor out sequencing related things into an independant new library: "evoral". 2008-09-19 00:47:49 +00:00
wscript Move waf up to top level, waf building of pbd, evoral, midi++ 2009-02-25 17:51:42 +00:00