I have committed your translation, thank you very much.

I will send a new demo and the rest of the updates later on this week.

That would be good, because the example no longer works now :/

Michael.

Hello,

While trying to work with the library, I have noticed that the packages/extra/sndfile/sndfile.pp is old, and most of the API there is obsolete and no longer exists on the library.

So I have created (from scratch) a new translation for the latest version (1.0.17) that came out over a year ago.

I will send a new demo and the rest of the updates later on this week.

Thanks,
Ido

That's a coincidence ! I have an updated sndfile.pp also, created from scratch and breaking the demo program. I uploaded it to <http://www.microbizz.nl/sndfile.pas.bz2>. You may want to have a compiles with both FPC and GPC.

Regards,

Adriaan van Os

Hi
Sorry for the late response.
First of all thank you Michael.

The reasons are: It uses the Linux unit, and while we need to convert it to the Unix units.. and then add the annoying prefix of "fp". Secondly, Some of the API that was used in the Demo is obsolete, so we can't really use it. even if you will try to use it with the old library, it will not be linked.

I uploaded it to <http://www.microbizz.nl/sndfile.pas.bz2>. You may

want to have a

compiles with both FPC and GPC.

Yes, some of your code is much better then mine :( Making it more general to be used. However you are missing pointers for the records (for example), and less Pascal way of coding style, such as adding T before typed declarations as an example.
However, your code is more general... If you allow me, I would add such additions from what you have made to the library.

Regards, Adriaan van Os

Thanks,
Ido

Another issue is that neither the Readme file nor sndfile.pp mentiones that sndfile is licensed under the LGPL <http://www.mega-nerd.com/libsndfile/>. Therefore, I think the FPC distribution violates the libsndfile LGPL license, as FPC itself is distributed under the GPL-with-linking exception.

Regards,

Adriaan van Os

P.S.
I suggest to rename the package directory to "libsndfile" as that seems to be the project's name <http://www.mega-nerd.com/libsndfile/>.

ik wrote

Yes, some of your code is much better then mine :( Making it more general to be used. However you are missing pointers for the records (for example), and less Pascal way of coding style, such as adding T before typed declarations as an example.

I question if that is "Pascal way of coding style", although it certainly is "Delphi way of coding style".

However, your code is more general... If you allow me, I would add such additions from what you have made to the library.

Please go ahead.

Regards,

Adriaan van Os

Copying from the header I have translated:

{
** Copyright (C) 1999-2006 Erik de Castro Lopo
**
** This program is free software; you can redistribute it and/or modify ** it under the terms of the GNU Lesser General Public License as published by ** the Free Software Foundation; either version 2.1 of the License, or ** (at your option) any later version.
**
** This program is distributed in the hope that it will be useful, ** but WITHOUT ANY WARRANTY; without even the implied warranty of ** MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the ** GNU Lesser General Public License for more details. **
** You should have received a copy of the GNU Lesser General Public License ** along with this program; if not, write to the Free Software ** Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. ...

So the source file does mention that it is LGPL. I fully translated the header as-is.

I didn't translate it because of "backward computability" with the old name. At first I thought on doing that. I believe that it was named like so, because of the header name.

Ido

1. The file currently in subversion does mentions the license.

2. We provide an interface to the library, not the library itself. The interface need not have the same license as the library.

Michael.

ik wrote

So the source file does mention that it is LGPL. I fully translated the header as-is.

Then my comment is no longer valid.

Regards,

Adriaan van Os

Hi,

Here is my latest updates including the example.
The example have some issues that I'm unable to understand yet, but it seems that because the code works faster then the C code, it does not write all of the content back to the device. I will try to solve this later on.

I fixed a bug and and made the library a bit more cross platfrom/compiler as Adrian did, but a bit different :)

Thanks,

Ido

I think that repeated use of

	{$IFDEF FPC} cdecl; {$ENDIF}

is unnecessary, because of the

{$IFDEF FPC}
   {$MODE FPC}
   {$CALLING cdecl}
   {$PACKRECORDS C}
{$ENDIF}

Thanks,

Thanks also,

Adriaan van Os