Re: [PATCH 0/2] Bse:: namespace fixes in bse includes



Hey Stefan,

thanks for sending the changes in.

But I think the patches are now obsolete, can you confirm?

Until everything is moved into the Bse namespace, sfiwrapper.hh does
using Bse::uint8; for all sorts of integer types, which should fix the
below mentioned issues.



On 07.04.2017 12:22, Stefan Westerfeld wrote:
I've meanwhile updated SpectMorph so that many parts of it no longer require
libbse (for instance wav i/o is now libsndfiles, and not GslDataHandle based).

Especially, SpectMorph now defines its own uint8, ... uint64 types and no
longer depends on libbse defining these types. After this update, there are
only three cases where bse includes need to be changed to build SpectMorph
against beast master. These are in the first patch.

The second patch corrects other instances of uint8, ... uint64 types referenced
in bse includes that are not used by SpectMorph, I don't need them to build
SpectMorph, but include them for the sake of completeness.


Stefan Westerfeld (2):
  BSE: add proper Bse:: namespace to uint8 and uint64 in bse includes
  BSE: more Bse:: namespace for uint8, uint32 and uint64 in bse includes

 bse/bsecategories.hh  |  2 +-
 bse/bseexports.hh     |  2 +-
 bse/bsemididecoder.hh | 12 ++++++------
 bse/bsemidievent.hh   |  6 +++---
 bse/bsepcmwriter.hh   | 10 +++++-----
 bse/bseutils.hh       |  2 +-
 6 files changed, 17 insertions(+), 17 deletions(-)


-- 
Yours sincerely,
Tim Janik

https://testbit.eu/timj/
Free software author.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]