Hi!
I think the best way to find a new name is to see how the #include file is
used:
$ grep -r bse-internals.hh $(find . -iname '*.hh' -o -iname '*.cc')
./tests/latency/bselatencytest.genidl.hh:#include <bse/bse-internals.hh>
./tests/bse/testplugin.genidl.hh:#include <bse/bse-internals.hh>
./sfi/sfidl-corecxx.cc: printf ("\n#include <bse/bse-internals.hh>\n");
./bse/bseprobe.genidl.hh:#include <bse/bse-internals.hh>
./bse/bsebusmodule.genidl.hh:#include <bse/bse-internals.hh>
./bse/bsebasics.genidl.hh:#include <bse/bse-internals.hh>
./plugins/davchorus.genidl.hh:#include <bse/bse-internals.hh>
./plugins/standardguspatchenvelope.genidl.hh:#include <bse/bse-internals.hh>
./plugins/davbassfilter.genidl.hh:#include <bse/bse-internals.hh>
./plugins/bsebalance.genidl.hh:#include <bse/bse-internals.hh>
./plugins/bseamplifier.genidl.hh:#include <bse/bse-internals.hh>
./plugins/artscompressor.genidl.hh:#include <bse/bse-internals.hh>
./plugins/bsesummation.genidl.hh:#include <bse/bse-internals.hh>
./plugins/bsequantizer.genidl.hh:#include <bse/bse-internals.hh>
./plugins/davorgan.genidl.hh:#include <bse/bse-internals.hh>
./plugins/bsenoise.genidl.hh:#include <bse/bse-internals.hh>
./plugins/bsecontribsampleandhold.genidl.hh:#include <bse/bse-internals.hh>
./plugins/standardsaturator.genidl.hh:#include <bse/bse-internals.hh>
So sfidl automatically includes this file for the C++ code generated from the
plugin IDL file. This is also the reason why SpectMorph needs this file, as it
contains IDL files and corresponding plugins. Maybe we could call it
bsepluginbase.hh.
Cu.. Stefan
On Tue, Oct 20, 2015 at 01:07:37AM +0200, Tim Janik wrote:
> an "internal" header like bse-internals.hh should obviously not be installed.
> Looking at it, it's mostly including otherwise public math headers though.
> So I winder why it got named like this in the first place. I guess the header
> should be renamed when installed, care to dig up its uses/history and
> suggest a better name?
>
>
> On 18.10.2015 16:42, Stefan Westerfeld wrote:
> > Signed-off-by: Stefan Westerfeld <[email protected]>
> > ---
> > bse/Makefile.am | 3 +--
> > 1 file changed, 1 insertion(+), 2 deletions(-)
> >
> > diff --git a/bse/Makefile.am b/bse/Makefile.am
> > index 3668938..76c2a09 100644
> > --- a/bse/Makefile.am
> > +++ b/bse/Makefile.am
> > @@ -33,7 +33,7 @@ bse_public_headers = $(strip \
> > bsebus.hh bsecategories.hh \
> > bsefilter.hh bsebiquadfilter.hh \
> > bseconstant.hh bseconstvalues.hh bsecontainer.hh
> > bsecontextmerger.hh \
> > - bseclientapi.hh \
> > + bseclientapi.hh bseserverapi.hh bse-internals.hh
> > \
> > bseeditablesample.hh bseenums.hh
> > bsegconfig.hh \
> > bseglobals.hh bseglue.hh bseitem.hh
> > bsejanitor.hh \
> > bsemain.hh bsemath.hh bsemathsignal.hh
> > bseladspa.hh \
> > @@ -129,7 +129,6 @@ EXTRA_DIST += $(strip \
> > bsewave.header \
> > gsl-fftgen.pl gsl-fftconf.sh gsloscillator-aux.cc \
> > gslincluder.hh gslwaveosc-aux.cc \
> > - bse-internals.hh \
> > )
> >
> > #
>
>
> --
> Yours sincerely,
> Tim Janik
>
> https://testbit.eu/timj/
> Free software author and speaker.
>
> _______________________________________________
> beast mailing list
> [email protected]
> https://mail.gnome.org/mailman/listinfo/beast
--
Stefan Westerfeld, http://space.twc.de/~stefan
_______________________________________________
beast mailing list
[email protected]
https://mail.gnome.org/mailman/listinfo/beast
