Csound Csound-dev Csound-tekno Search About

[Cs-dev] sndfile.h on ppc?

Date2010-07-27 01:51
Frommatt ingalls
Subject[Cs-dev] sndfile.h on ppc?
im having trouble compile an opcode on a PPC mac.
getting an error about  isn't found.

this is a libsndfile file, right?

i found this on my intel mac in /usr/local/include 
but that directory isnt even on my ppc machine.

any ideas? could it be that the 5.12UB install isn't right?

the reason i am even doing this is a noticed the dylib isnt loading on ppc 
(even though i am compiling it as a ppc and runs fine under rosetta on my intel machine -- weird.)

thanks,
matt
------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
of $1 Million in cash or HP Products. Visit us here for more details:
http://ad.doubleclick.net/clk;226879339;13503038;l?
http://clk.atdmt.com/CRS/go/247765532/direct/01/
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2010-07-27 02:12
Frommatt ingalls
SubjectRe: [Cs-dev] sndfile.h on ppc?
well copying /usr/local/include from my intel machine to my ppc machine seemed to fix the problem.
perhaps it's a bug in the libsndfile installer?

On Jul 26, 2010, at 5:51 PM, matt ingalls wrote:

> im having trouble compile an opcode on a PPC mac.
> getting an error about  isn't found.
> 
> this is a libsndfile file, right?
> 
> i found this on my intel mac in /usr/local/include 
> but that directory isnt even on my ppc machine.
> 
> any ideas? could it be that the 5.12UB install isn't right?
> 
> the reason i am even doing this is a noticed the dylib isnt loading on ppc 
> (even though i am compiling it as a ppc and runs fine under rosetta on my intel machine -- weird.)
> 
> thanks,
> matt
> ------------------------------------------------------------------------------
> The Palm PDK Hot Apps Program offers developers who use the
> Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
> of $1 Million in cash or HP Products. Visit us here for more details:
> http://ad.doubleclick.net/clk;226879339;13503038;l?
> http://clk.atdmt.com/CRS/go/247765532/direct/01/
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel


------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
of $1 Million in cash or HP Products. Visit us here for more details:
http://ad.doubleclick.net/clk;226879339;13503038;l?
http://clk.atdmt.com/CRS/go/247765532/direct/01/
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2010-07-27 04:51
FromErik de Castro Lopo
SubjectRe: [Cs-dev] sndfile.h on ppc?
matt ingalls wrote:

> well copying /usr/local/include from my intel machine to my ppc
> machine seemed to fix the problem.

It works in this case, but in general, that is not a good idea.

It would be perfectly valid for a header like that to be
autogenerated for its intended platform so that when you 
copy it from intel to powerpc, you end up with an incorrect
header file.

Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/

------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
of $1 Million in cash or HP Products. Visit us here for more details:
http://ad.doubleclick.net/clk;226879339;13503038;l?
http://clk.atdmt.com/CRS/go/247765532/direct/01/
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2010-07-27 10:44
FromVictor Lazzarini
SubjectRe: [Cs-dev] sndfile.h on ppc?
The Csound installer does not install the libsndfile header, only the  
dylib. If you are developing, then you can just build and install  
libsndfile, which is simple enough.
I need to investigate this, but I think that  shouldn't really  
end up including sndfile.h. There might a better of way of placing  
that header in the Csound sources so that plugins do not need to have  
it. After all, no plugin should ever call a libsndfile function  
directly.

Victor

On 27 Jul 2010, at 04:51, Erik de Castro Lopo wrote:

> matt ingalls wrote:
>
>> well copying /usr/local/include from my intel machine to my ppc
>> machine seemed to fix the problem.
>
> It works in this case, but in general, that is not a good idea.
>
> It would be perfectly valid for a header like that to be
> autogenerated for its intended platform so that when you
> copy it from intel to powerpc, you end up with an incorrect
> header file.
>
> Erik
> -- 
> ----------------------------------------------------------------------
> Erik de Castro Lopo
> http://www.mega-nerd.com/
>
> ------------------------------------------------------------------------------
> The Palm PDK Hot Apps Program offers developers who use the
> Plug-In Development Kit to bring their C/C++ apps to Palm for a share
> of $1 Million in cash or HP Products. Visit us here for more details:
> http://ad.doubleclick.net/clk;226879339;13503038;l?
> http://clk.atdmt.com/CRS/go/247765532/direct/01/
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel


------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
of $1 Million in cash or HP Products. Visit us here for more details:
http://ad.doubleclick.net/clk;226879339;13503038;l?
http://clk.atdmt.com/CRS/go/247765532/direct/01/
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2010-07-27 18:09
Frommatt ingalls
SubjectRe: [Cs-dev] sndfile.h on ppc?
ah - ok thanks.  

i thought i installed libsndfile directly on both machines, but
maybe not...  

it does seem like sndfile.h is needed for some struct definitions -
would be nice to get rid of that requirement :)

-m

On Jul 27, 2010, at 2:44 AM, Victor Lazzarini wrote:

> The Csound installer does not install the libsndfile header, only the  
> dylib. If you are developing, then you can just build and install  
> libsndfile, which is simple enough.
> I need to investigate this, but I think that  shouldn't really  
> end up including sndfile.h. There might a better of way of placing  
> that header in the Csound sources so that plugins do not need to have  
> it. After all, no plugin should ever call a libsndfile function  
> directly.
> 
> Victor
> 
> On 27 Jul 2010, at 04:51, Erik de Castro Lopo wrote:
> 
>> matt ingalls wrote:
>> 
>>> well copying /usr/local/include from my intel machine to my ppc
>>> machine seemed to fix the problem.
>> 
>> It works in this case, but in general, that is not a good idea.
>> 
>> It would be perfectly valid for a header like that to be
>> autogenerated for its intended platform so that when you
>> copy it from intel to powerpc, you end up with an incorrect
>> header file.
>> 
>> Erik
>> -- 
>> ----------------------------------------------------------------------
>> Erik de Castro Lopo
>> http://www.mega-nerd.com/
>> 
>> ------------------------------------------------------------------------------
>> The Palm PDK Hot Apps Program offers developers who use the
>> Plug-In Development Kit to bring their C/C++ apps to Palm for a share
>> of $1 Million in cash or HP Products. Visit us here for more details:
>> http://ad.doubleclick.net/clk;226879339;13503038;l?
>> http://clk.atdmt.com/CRS/go/247765532/direct/01/
>> _______________________________________________
>> Csound-devel mailing list
>> Csound-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/csound-devel
> 
> 
> ------------------------------------------------------------------------------
> The Palm PDK Hot Apps Program offers developers who use the
> Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
> of $1 Million in cash or HP Products. Visit us here for more details:
> http://ad.doubleclick.net/clk;226879339;13503038;l?
> http://clk.atdmt.com/CRS/go/247765532/direct/01/
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel


------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
of $1 Million in cash or HP Products. Visit us here for more details:
http://ad.doubleclick.net/clk;226879339;13503038;l?
http://clk.atdmt.com/CRS/go/247765532/direct/01/
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2010-07-27 18:37
FromVictor Lazzarini
SubjectRe: [Cs-dev] sndfile.h on ppc?
Definitely. I'm on the job...
On 27 Jul 2010, at 18:09, matt ingalls wrote:

> it does seem like sndfile.h is needed for some struct definitions -
> would be nice to get rid of that requirement :)


------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
of $1 Million in cash or HP Products. Visit us here for more details:
http://ad.doubleclick.net/clk;226879339;13503038;l?
http://clk.atdmt.com/CRS/go/247765532/direct/01/
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2010-07-27 19:44
FromVictor Lazzarini
Subject[Cs-dev] sndfile.h requirement for plugins (was Re: sndfile.h on ppc?)
I have removed the sndifle.h header include line from csoundCore.h to  
allow opcodes to be built without libsndfile headers

Some  changes were needed. As it turned out only two CSOUND functions  
used sndfile.h data structures, so I think it was worth a go. This was  
what I changed,
I believe there is no need for but a minor API bump:

SNDMEMFILE *csoundLoadSoundFile(CSOUND *csound, const char *fileName,
                                                  void *sfinfo_)
last argument changed from SF_INFO * to void *

void    rewriteheader(void *ofd);
argument changed from SNDFILE * to void *

Changes implemented in Engine/memfiles.c and InOut/libsnd_u.c

The libsndfile header is now place only where it is used:
Engine/envvar.c
Engine/memfiles.c
Engine/libsnd_u.c
OOps/sndinfUG.c
Opcodes/fout.c
util/atsa.c
Opcodes/stdopcode.h
H/diskin2.h
H/soundio.h
util/pvanal.c
util/sndinfo.c
util/xtrct.c

===========================

If there are no complaints I will commit this tomorrow morning.

Victor


On 27 Jul 2010, at 18:37, Victor Lazzarini wrote:

> Definitely. I'm on the job...
> On 27 Jul 2010, at 18:09, matt ingalls wrote:
>
>> it does seem like sndfile.h is needed for some struct definitions -
>> would be nice to get rid of that requirement :)
>
>
> ------------------------------------------------------------------------------
> The Palm PDK Hot Apps Program offers developers who use the
> Plug-In Development Kit to bring their C/C++ apps to Palm for a share
> of $1 Million in cash or HP Products. Visit us here for more details:
> http://ad.doubleclick.net/clk;226879339;13503038;l?
> http://clk.atdmt.com/CRS/go/247765532/direct/01/
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel


------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
of $1 Million in cash or HP Products. Visit us here for more details:
http://ad.doubleclick.net/clk;226879339;13503038;l?
http://clk.atdmt.com/CRS/go/247765532/direct/01/
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2010-07-27 23:58
FromErik de Castro Lopo
SubjectRe: [Cs-dev] sndfile.h requirement for plugins (was Re: sndfile.h on ppc?)
Victor Lazzarini wrote:

> I have removed the sndifle.h header include line from csoundCore.h to  
> allow opcodes to be built without libsndfile headers
> 
> Some  changes were needed. As it turned out only two CSOUND functions  
> used sndfile.h data structures, so I think it was worth a go. This was  
> what I changed,
> I believe there is no need for but a minor API bump:
> 
> SNDMEMFILE *csoundLoadSoundFile(CSOUND *csound, const char *fileName,
>                                                   void *sfinfo_)
> last argument changed from SF_INFO * to void *
> 
> void    rewriteheader(void *ofd);
> argument changed from SNDFILE * to void *

There is a better way to do this. You can forward declare the
struct using:

    struct SF_INFO ;

and the change the

     void *sfinfo_

to

     struct SFINFO * sfinfo_

Similar for the SNDFILE pointer.

Casting from void pointer to structs makes my skin crawl.

Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/

------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
of $1 Million in cash or HP Products. Visit us here for more details:
http://ad.doubleclick.net/clk;226879339;13503038;l?
http://clk.atdmt.com/CRS/go/247765532/direct/01/
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2010-07-28 00:09
FromVictor Lazzarini
SubjectRe: [Cs-dev] sndfile.h requirement for plugins (was Re: sndfile.h on ppc?)
Ok, thanks.

Victor
On 27 Jul 2010, at 23:58, Erik de Castro Lopo wrote:

> Victor Lazzarini wrote:
>
>> I have removed the sndifle.h header include line from csoundCore.h to
>> allow opcodes to be built without libsndfile headers
>>
>> Some  changes were needed. As it turned out only two CSOUND functions
>> used sndfile.h data structures, so I think it was worth a go. This  
>> was
>> what I changed,
>> I believe there is no need for but a minor API bump:
>>
>> SNDMEMFILE *csoundLoadSoundFile(CSOUND *csound, const char *fileName,
>>                                                  void *sfinfo_)
>> last argument changed from SF_INFO * to void *
>>
>> void    rewriteheader(void *ofd);
>> argument changed from SNDFILE * to void *
>
> There is a better way to do this. You can forward declare the
> struct using:
>
>    struct SF_INFO ;
>
> and the change the
>
>     void *sfinfo_
>
> to
>
>     struct SFINFO * sfinfo_
>
> Similar for the SNDFILE pointer.
>
> Casting from void pointer to structs makes my skin crawl.
>
> Erik
> -- 
> ----------------------------------------------------------------------
> Erik de Castro Lopo
> http://www.mega-nerd.com/
>
> ------------------------------------------------------------------------------
> The Palm PDK Hot Apps Program offers developers who use the
> Plug-In Development Kit to bring their C/C++ apps to Palm for a share
> of $1 Million in cash or HP Products. Visit us here for more details:
> http://ad.doubleclick.net/clk;226879339;13503038;l?
> http://clk.atdmt.com/CRS/go/247765532/direct/01/
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel


------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
of $1 Million in cash or HP Products. Visit us here for more details:
http://ad.doubleclick.net/clk;226879339;13503038;l?
http://clk.atdmt.com/CRS/go/247765532/direct/01/
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2010-07-28 11:28
FromAndres Cabrera
SubjectRe: [Cs-dev] sndfile.h requirement for plugins (was Re: sndfile.h on ppc?)
I don't know about this, so this may be a silly question, but would
this have any effect on binary compatibility of plugins or front-ends?

Cheers,
Andrés

On Wed, Jul 28, 2010 at 12:09 AM, Victor Lazzarini
 wrote:
> Ok, thanks.
>
> Victor
> On 27 Jul 2010, at 23:58, Erik de Castro Lopo wrote:
>
>> Victor Lazzarini wrote:
>>
>>> I have removed the sndifle.h header include line from csoundCore.h to
>>> allow opcodes to be built without libsndfile headers
>>>
>>> Some  changes were needed. As it turned out only two CSOUND functions
>>> used sndfile.h data structures, so I think it was worth a go. This
>>> was
>>> what I changed,
>>> I believe there is no need for but a minor API bump:
>>>
>>> SNDMEMFILE *csoundLoadSoundFile(CSOUND *csound, const char *fileName,
>>>                                                  void *sfinfo_)
>>> last argument changed from SF_INFO * to void *
>>>
>>> void    rewriteheader(void *ofd);
>>> argument changed from SNDFILE * to void *
>>
>> There is a better way to do this. You can forward declare the
>> struct using:
>>
>>    struct SF_INFO ;
>>
>> and the change the
>>
>>     void *sfinfo_
>>
>> to
>>
>>     struct SFINFO * sfinfo_
>>
>> Similar for the SNDFILE pointer.
>>
>> Casting from void pointer to structs makes my skin crawl.
>>
>> Erik
>> --
>> ----------------------------------------------------------------------
>> Erik de Castro Lopo
>> http://www.mega-nerd.com/
>>
>> ------------------------------------------------------------------------------
>> The Palm PDK Hot Apps Program offers developers who use the
>> Plug-In Development Kit to bring their C/C++ apps to Palm for a share
>> of $1 Million in cash or HP Products. Visit us here for more details:
>> http://ad.doubleclick.net/clk;226879339;13503038;l?
>> http://clk.atdmt.com/CRS/go/247765532/direct/01/
>> _______________________________________________
>> Csound-devel mailing list
>> Csound-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>
>
> ------------------------------------------------------------------------------
> The Palm PDK Hot Apps Program offers developers who use the
> Plug-In Development Kit to bring their C/C++ apps to Palm for a share
> of $1 Million in cash or HP Products. Visit us here for more details:
> http://ad.doubleclick.net/clk;226879339;13503038;l?
> http://clk.atdmt.com/CRS/go/247765532/direct/01/
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel
>



-- 


Andrés

------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
of $1 Million in cash or HP Products. Visit us here for more details:
http://ad.doubleclick.net/clk;226879339;13503038;l?
http://clk.atdmt.com/CRS/go/247765532/direct/01/
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo

Date2010-07-28 11:57
FromVictor Lazzarini
SubjectRe: [Cs-dev] sndfile.h requirement for plugins (was Re: sndfile.h on ppc?)
I don't think so, because the sndfile header in csoundCore.h was only  
there to provide the definitions for these two structure pointers. The  
replacement now with Erik's suggestion should be seamless and no API  
bump of any form is needed.

Victor

On 28 Jul 2010, at 11:28, Andres Cabrera wrote:

> I don't know about this, so this may be a silly question, but would
> this have any effect on binary compatibility of plugins or front-ends?
>
> Cheers,
> Andrés
>
> On Wed, Jul 28, 2010 at 12:09 AM, Victor Lazzarini
>  wrote:
>> Ok, thanks.
>>
>> Victor
>> On 27 Jul 2010, at 23:58, Erik de Castro Lopo wrote:
>>
>>> Victor Lazzarini wrote:
>>>
>>>> I have removed the sndifle.h header include line from  
>>>> csoundCore.h to
>>>> allow opcodes to be built without libsndfile headers
>>>>
>>>> Some  changes were needed. As it turned out only two CSOUND  
>>>> functions
>>>> used sndfile.h data structures, so I think it was worth a go. This
>>>> was
>>>> what I changed,
>>>> I believe there is no need for but a minor API bump:
>>>>
>>>> SNDMEMFILE *csoundLoadSoundFile(CSOUND *csound, const char  
>>>> *fileName,
>>>>                                                 void *sfinfo_)
>>>> last argument changed from SF_INFO * to void *
>>>>
>>>> void    rewriteheader(void *ofd);
>>>> argument changed from SNDFILE * to void *
>>>
>>> There is a better way to do this. You can forward declare the
>>> struct using:
>>>
>>>   struct SF_INFO ;
>>>
>>> and the change the
>>>
>>>    void *sfinfo_
>>>
>>> to
>>>
>>>    struct SFINFO * sfinfo_
>>>
>>> Similar for the SNDFILE pointer.
>>>
>>> Casting from void pointer to structs makes my skin crawl.
>>>
>>> Erik
>>> --
>>> ----------------------------------------------------------------------
>>> Erik de Castro Lopo
>>> http://www.mega-nerd.com/
>>>
>>> ------------------------------------------------------------------------------
>>> The Palm PDK Hot Apps Program offers developers who use the
>>> Plug-In Development Kit to bring their C/C++ apps to Palm for a  
>>> share
>>> of $1 Million in cash or HP Products. Visit us here for more  
>>> details:
>>> http://ad.doubleclick.net/clk;226879339;13503038;l?
>>> http://clk.atdmt.com/CRS/go/247765532/direct/01/
>>> _______________________________________________
>>> Csound-devel mailing list
>>> Csound-devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>>
>>
>> ------------------------------------------------------------------------------
>> The Palm PDK Hot Apps Program offers developers who use the
>> Plug-In Development Kit to bring their C/C++ apps to Palm for a share
>> of $1 Million in cash or HP Products. Visit us here for more details:
>> http://ad.doubleclick.net/clk;226879339;13503038;l?
>> http://clk.atdmt.com/CRS/go/247765532/direct/01/
>> _______________________________________________
>> Csound-devel mailing list
>> Csound-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>>
>
>
>
> -- 
>
>
> Andrés
>
> ------------------------------------------------------------------------------
> The Palm PDK Hot Apps Program offers developers who use the
> Plug-In Development Kit to bring their C/C++ apps to Palm for a share
> of $1 Million in cash or HP Products. Visit us here for more details:
> http://ad.doubleclick.net/clk;226879339;13503038;l?
> http://clk.atdmt.com/CRS/go/247765532/direct/01/
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel


------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
of $1 Million in cash or HP Products. Visit us here for more details:
http://ad.doubleclick.net/clk;226879339;13503038;l?
http://clk.atdmt.com/CRS/go/247765532/direct/01/
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2010-07-28 12:05
FromAndres Cabrera
SubjectRe: [Cs-dev] sndfile.h requirement for plugins (was Re: sndfile.h on ppc?)
Great! Thanks for the info.

Cheers,
Andrés

On Wed, Jul 28, 2010 at 11:57 AM, Victor Lazzarini
 wrote:
> I don't think so, because the sndfile header in csoundCore.h was only
> there to provide the definitions for these two structure pointers. The
> replacement now with Erik's suggestion should be seamless and no API
> bump of any form is needed.
>
> Victor
>
> On 28 Jul 2010, at 11:28, Andres Cabrera wrote:
>
>> I don't know about this, so this may be a silly question, but would
>> this have any effect on binary compatibility of plugins or front-ends?
>>
>> Cheers,
>> Andrés
>>
>> On Wed, Jul 28, 2010 at 12:09 AM, Victor Lazzarini
>>  wrote:
>>> Ok, thanks.
>>>
>>> Victor
>>> On 27 Jul 2010, at 23:58, Erik de Castro Lopo wrote:
>>>
>>>> Victor Lazzarini wrote:
>>>>
>>>>> I have removed the sndifle.h header include line from
>>>>> csoundCore.h to
>>>>> allow opcodes to be built without libsndfile headers
>>>>>
>>>>> Some  changes were needed. As it turned out only two CSOUND
>>>>> functions
>>>>> used sndfile.h data structures, so I think it was worth a go. This
>>>>> was
>>>>> what I changed,
>>>>> I believe there is no need for but a minor API bump:
>>>>>
>>>>> SNDMEMFILE *csoundLoadSoundFile(CSOUND *csound, const char
>>>>> *fileName,
>>>>>                                                 void *sfinfo_)
>>>>> last argument changed from SF_INFO * to void *
>>>>>
>>>>> void    rewriteheader(void *ofd);
>>>>> argument changed from SNDFILE * to void *
>>>>
>>>> There is a better way to do this. You can forward declare the
>>>> struct using:
>>>>
>>>>   struct SF_INFO ;
>>>>
>>>> and the change the
>>>>
>>>>    void *sfinfo_
>>>>
>>>> to
>>>>
>>>>    struct SFINFO * sfinfo_
>>>>
>>>> Similar for the SNDFILE pointer.
>>>>
>>>> Casting from void pointer to structs makes my skin crawl.
>>>>
>>>> Erik
>>>> --
>>>> ----------------------------------------------------------------------
>>>> Erik de Castro Lopo
>>>> http://www.mega-nerd.com/
>>>>
>>>> ------------------------------------------------------------------------------
>>>> The Palm PDK Hot Apps Program offers developers who use the
>>>> Plug-In Development Kit to bring their C/C++ apps to Palm for a
>>>> share
>>>> of $1 Million in cash or HP Products. Visit us here for more
>>>> details:
>>>> http://ad.doubleclick.net/clk;226879339;13503038;l?
>>>> http://clk.atdmt.com/CRS/go/247765532/direct/01/
>>>> _______________________________________________
>>>> Csound-devel mailing list
>>>> Csound-devel@lists.sourceforge.net
>>>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>>>
>>>
>>> ------------------------------------------------------------------------------
>>> The Palm PDK Hot Apps Program offers developers who use the
>>> Plug-In Development Kit to bring their C/C++ apps to Palm for a share
>>> of $1 Million in cash or HP Products. Visit us here for more details:
>>> http://ad.doubleclick.net/clk;226879339;13503038;l?
>>> http://clk.atdmt.com/CRS/go/247765532/direct/01/
>>> _______________________________________________
>>> Csound-devel mailing list
>>> Csound-devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>>>
>>
>>
>>
>> --
>>
>>
>> Andrés
>>
>> ------------------------------------------------------------------------------
>> The Palm PDK Hot Apps Program offers developers who use the
>> Plug-In Development Kit to bring their C/C++ apps to Palm for a share
>> of $1 Million in cash or HP Products. Visit us here for more details:
>> http://ad.doubleclick.net/clk;226879339;13503038;l?
>> http://clk.atdmt.com/CRS/go/247765532/direct/01/
>> _______________________________________________
>> Csound-devel mailing list
>> Csound-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>
>
> ------------------------------------------------------------------------------
> The Palm PDK Hot Apps Program offers developers who use the
> Plug-In Development Kit to bring their C/C++ apps to Palm for a share
> of $1 Million in cash or HP Products. Visit us here for more details:
> http://ad.doubleclick.net/clk;226879339;13503038;l?
> http://clk.atdmt.com/CRS/go/247765532/direct/01/
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel
>



-- 


Andrés

------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
of $1 Million in cash or HP Products. Visit us here for more details:
http://ad.doubleclick.net/clk;226879339;13503038;l?
http://clk.atdmt.com/CRS/go/247765532/direct/01/
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net
ht

Date2010-07-28 12:16
FromVictor Lazzarini
SubjectRe: [Cs-dev] sndfile.h requirement for plugins (was Re: sndfile.h on ppc?)
OK, changes are committed to CVS now.

Victor
On 28 Jul 2010, at 12:05, Andres Cabrera wrote:

> Great! Thanks for the info.
>
> Cheers,
> Andrés
>
> On Wed, Jul 28, 2010 at 11:57 AM, Victor Lazzarini
>  wrote:
>> I don't think so, because the sndfile header in csoundCore.h was only
>> there to provide the definitions for these two structure pointers.  
>> The
>> replacement now with Erik's suggestion should be seamless and no API
>> bump of any form is needed.
>>
>> Victor
>>
>> On 28 Jul 2010, at 11:28, Andres Cabrera wrote:
>>
>>> I don't know about this, so this may be a silly question, but would
>>> this have any effect on binary compatibility of plugins or front- 
>>> ends?
>>>
>>> Cheers,
>>> Andrés
>>>
>>> On Wed, Jul 28, 2010 at 12:09 AM, Victor Lazzarini
>>>  wrote:
>>>> Ok, thanks.
>>>>
>>>> Victor
>>>> On 27 Jul 2010, at 23:58, Erik de Castro Lopo wrote:
>>>>
>>>>> Victor Lazzarini wrote:
>>>>>
>>>>>> I have removed the sndifle.h header include line from
>>>>>> csoundCore.h to
>>>>>> allow opcodes to be built without libsndfile headers
>>>>>>
>>>>>> Some  changes were needed. As it turned out only two CSOUND
>>>>>> functions
>>>>>> used sndfile.h data structures, so I think it was worth a go.  
>>>>>> This
>>>>>> was
>>>>>> what I changed,
>>>>>> I believe there is no need for but a minor API bump:
>>>>>>
>>>>>> SNDMEMFILE *csoundLoadSoundFile(CSOUND *csound, const char
>>>>>> *fileName,
>>>>>>                                                 void *sfinfo_)
>>>>>> last argument changed from SF_INFO * to void *
>>>>>>
>>>>>> void    rewriteheader(void *ofd);
>>>>>> argument changed from SNDFILE * to void *
>>>>>
>>>>> There is a better way to do this. You can forward declare the
>>>>> struct using:
>>>>>
>>>>>   struct SF_INFO ;
>>>>>
>>>>> and the change the
>>>>>
>>>>>    void *sfinfo_
>>>>>
>>>>> to
>>>>>
>>>>>    struct SFINFO * sfinfo_
>>>>>
>>>>> Similar for the SNDFILE pointer.
>>>>>
>>>>> Casting from void pointer to structs makes my skin crawl.
>>>>>
>>>>> Erik
>>>>> --
>>>>> ----------------------------------------------------------------------
>>>>> Erik de Castro Lopo
>>>>> http://www.mega-nerd.com/
>>>>>
>>>>> ------------------------------------------------------------------------------
>>>>> The Palm PDK Hot Apps Program offers developers who use the
>>>>> Plug-In Development Kit to bring their C/C++ apps to Palm for a
>>>>> share
>>>>> of $1 Million in cash or HP Products. Visit us here for more
>>>>> details:
>>>>> http://ad.doubleclick.net/clk;226879339;13503038;l?
>>>>> http://clk.atdmt.com/CRS/go/247765532/direct/01/
>>>>> _______________________________________________
>>>>> Csound-devel mailing list
>>>>> Csound-devel@lists.sourceforge.net
>>>>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>>>>
>>>>
>>>> ------------------------------------------------------------------------------
>>>> The Palm PDK Hot Apps Program offers developers who use the
>>>> Plug-In Development Kit to bring their C/C++ apps to Palm for a  
>>>> share
>>>> of $1 Million in cash or HP Products. Visit us here for more  
>>>> details:
>>>> http://ad.doubleclick.net/clk;226879339;13503038;l?
>>>> http://clk.atdmt.com/CRS/go/247765532/direct/01/
>>>> _______________________________________________
>>>> Csound-devel mailing list
>>>> Csound-devel@lists.sourceforge.net
>>>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>>>>
>>>
>>>
>>>
>>> --
>>>
>>>
>>> Andrés
>>>
>>> ------------------------------------------------------------------------------
>>> The Palm PDK Hot Apps Program offers developers who use the
>>> Plug-In Development Kit to bring their C/C++ apps to Palm for a  
>>> share
>>> of $1 Million in cash or HP Products. Visit us here for more  
>>> details:
>>> http://ad.doubleclick.net/clk;226879339;13503038;l?
>>> http://clk.atdmt.com/CRS/go/247765532/direct/01/
>>> _______________________________________________
>>> Csound-devel mailing list
>>> Csound-devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>>
>>
>> ------------------------------------------------------------------------------
>> The Palm PDK Hot Apps Program offers developers who use the
>> Plug-In Development Kit to bring their C/C++ apps to Palm for a share
>> of $1 Million in cash or HP Products. Visit us here for more details:
>> http://ad.doubleclick.net/clk;226879339;13503038;l?
>> http://clk.atdmt.com/CRS/go/247765532/direct/01/
>> _______________________________________________
>> Csound-devel mailing list
>> Csound-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>>
>
>
>
> -- 
>
>
> Andrés
>
> ------------------------------------------------------------------------------
> The Palm PDK Hot Apps Program offers developers who use the
> Plug-In Development Kit to bring their C/C++ apps to Palm for a share
> of $1 Million in cash or HP Products. Visit us here for more details:
> http://ad.doubleclick.net/clk;226879339;13503038;l?
> http://clk.atdmt.com/CRS/go/247765532/direct/01/
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel


------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share 
of $1 Million in cash or HP Products. Visit us here for more details:
http://ad.doubleclick.net/clk;226879339;13503038;l?
http://clk.atdmt.com/CRS/go/247765532/direct/01/
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net