Mercurial > hg > audiodb
diff audioDB.cpp @ 381:9742ea0ac33d
API const correctness.
The char *path arguments to audiodb_open() and audiodb_create() should
be const; make it so.
Also arrange for the datasize, ntracks and datadim arguments to
audiodb_create() to be unsigned to match the fields in the internal
audioDB class. Technically this is an ABI change, but since nothing is
calling this function with anything other than zero arguments yet
(correct me if I'm wrong) no-one should notice. (If you notice, shout)
author | mas01cr |
---|---|
date | Fri, 21 Nov 2008 14:32:27 +0000 |
parents | cfc55df37ceb |
children | 8425d05eda33 |
line wrap: on
line diff
--- a/audioDB.cpp Fri Nov 21 12:23:08 2008 +0000 +++ b/audioDB.cpp Fri Nov 21 14:32:27 2008 +0000 @@ -907,9 +907,7 @@ #include "audioDB_API.h" - - //adb_ptr audiodb_create(char * path,long ntracks, long datadim) { - adb_ptr audiodb_create(char * path,long datasize,long ntracks, long datadim) { + adb_ptr audiodb_create(const char *path, unsigned datasize, unsigned ntracks, unsigned datadim) { const char *argv[12]; int argvctr=0; char tempstr1[200]; @@ -925,19 +923,19 @@ if (datasize >0){ argv[argvctr++]="--datasize"; - snprintf(tempstr1,sizeof(tempstr1),"%ld",datasize); + snprintf(tempstr1,sizeof(tempstr1),"%u",datasize); argv[argvctr++]=tempstr1; } if (ntracks >0){ argv[argvctr++]="--ntracks"; - snprintf(tempstr2,sizeof(tempstr2),"%ld",ntracks); + snprintf(tempstr2,sizeof(tempstr2),"%u",ntracks); argv[argvctr++]=tempstr2; } if (datadim > 0){ argv[argvctr++]="--datadim"; - snprintf(tempstr3,sizeof(tempstr3),"%ld",datadim); + snprintf(tempstr3,sizeof(tempstr3),"%u",datadim); argv[argvctr++]=tempstr3; } @@ -1310,7 +1308,7 @@ return apierror; } - adb_ptr audiodb_open(char * path){ + adb_ptr audiodb_open(const char *path){ adb_ptr mydbp; adbstatus mystatus;