From ef6c500edb7dc1a4eb9bff11b83b57dd60d37d4a Mon Sep 17 00:00:00 2001 From: Warren Dukes Date: Fri, 4 Jun 2004 00:49:33 +0000 Subject: clean up ACK error coding a little bit git-svn-id: https://svn.musicpd.org/mpd/trunk@1321 09075e82-0dd4-0310-85a5-a0d7c8717e4f --- src/command.c | 59 +++++++++++++++++++----------------------------- src/command.h | 9 ++++---- src/directory.c | 17 +++++++------- src/interface.c | 14 +++++------- src/player.c | 3 +-- src/playlist.c | 70 +++++++++++++++++++++++++-------------------------------- src/tables.c | 9 ++++---- src/volume.c | 13 +++++------ 8 files changed, 83 insertions(+), 111 deletions(-) diff --git a/src/command.c b/src/command.c index 13325e851..121c9358e 100644 --- a/src/command.c +++ b/src/command.c @@ -151,7 +151,7 @@ int handlePlay(FILE * fp, unsigned int * permission, int argArrayLength, if(argArrayLength==2) { song = strtol(argArray[1],&test,10); if(*test!='\0') { - myfprintf(fp,"%s need a positive integer\n",COMMAND_RESPOND_ERROR); + commandError(fp, "need a positive integer"); return -1; } } @@ -171,8 +171,7 @@ int handlePause(FILE * fp, unsigned int * permission, char * test; int pause = strtol(argArray[1],&test,10); if(*test!='\0' || (pause!=0 && pause!=1)) { - myfprintf(fp,"%s \%s\" is not 0 or 1\n", - COMMAND_RESPOND_ERROR,pause); + commandError(fp, "\%s\" is not 0 or 1", argArray[1]); return -1; } return playerSetPause(fp,pause); @@ -263,7 +262,7 @@ int handleDelete(FILE * fp, unsigned int * permission, int argArrayLength, song = strtol(argArray[1],&test,10); if(*test!='\0') { - myfprintf(fp,"%s need a positive integer\n",COMMAND_RESPOND_ERROR); + commandError(fp, "need a positive integer"); return -1; } return deleteFromPlaylist(fp,song); @@ -327,7 +326,7 @@ int handlePlaylistInfo(FILE * fp, unsigned int * permission, if(argArrayLength == 2) { song = strtol(argArray[1],&test,10); if(*test!='\0') { - myfprintf(fp,"%s need a positive integer\n",COMMAND_RESPOND_ERROR); + commandError(fp, "%s need a positive integer"); return -1; } } @@ -416,7 +415,7 @@ int handleVolume(FILE * fp, unsigned int * permission, int argArrayLength, change = strtol(argArray[1],&test,10); if(*test!='\0') { - myfprintf(fp,"%s need an integer\n",COMMAND_RESPOND_ERROR); + commandError(fp, "need an integer"); return -1; } return changeVolumeLevel(fp,change,1); @@ -430,7 +429,7 @@ int handleSetVol(FILE * fp, unsigned int * permission, int argArrayLength, level = strtol(argArray[1],&test,10); if(*test!='\0') { - myfprintf(fp,"%s need an integer\n",COMMAND_RESPOND_ERROR); + commandError(fp, "need an integer"); return -1; } return changeVolumeLevel(fp,level,0); @@ -444,7 +443,7 @@ int handleRepeat(FILE * fp, unsigned int * permission, int argArrayLength, status = strtol(argArray[1],&test,10); if(*test!='\0') { - myfprintf(fp,"%s need an integer\n",COMMAND_RESPOND_ERROR); + commandError(fp, "need an integer"); return -1; } return setPlaylistRepeatStatus(fp,status); @@ -458,7 +457,7 @@ int handleRandom(FILE * fp, unsigned int * permission, int argArrayLength, status = strtol(argArray[1],&test,10); if(*test!='\0') { - myfprintf(fp,"%s need an integer\n",COMMAND_RESPOND_ERROR); + commandError(fp, "need an integer"); return -1; } return setPlaylistRandomStatus(fp,status); @@ -495,14 +494,12 @@ int handleMove(FILE * fp, unsigned int * permission, int argArrayLength, from = strtol(argArray[1],&test,10); if(*test!='\0') { - myfprintf(fp,"%s \"%s\" is not a integer\n", - COMMAND_RESPOND_ERROR,argArray[1]); + commandError(fp, "\"%s\" is not a integer", argArray[1]); return -1; } to = strtol(argArray[2],&test,10); if(*test!='\0') { - myfprintf(fp,"%s \"%s\" is not a integer\n", - COMMAND_RESPOND_ERROR,argArray[2]); + commandError(fp, "\"%s\" is not a integer", argArray[2]); return -1; } return moveSongInPlaylist(fp,from,to); @@ -517,14 +514,12 @@ int handleSwap(FILE * fp, unsigned int * permission, int argArrayLength, song1 = strtol(argArray[1],&test,10); if(*test!='\0') { - myfprintf(fp,"%s \"%s\" is not a integer\n", - COMMAND_RESPOND_ERROR,argArray[1]); + commandError(fp, "\"%s\" is not a integer", argArray[1]); return -1; } song2 = strtol(argArray[2],&test,10); if(*test!='\0') { - myfprintf(fp,"%s \"%s\" is not a integer\n", - COMMAND_RESPOND_ERROR,argArray[2]); + commandError(fp, "\"%s\" is not a integer", argArray[2]); return -1; } return swapSongsInPlaylist(fp,song1,song2); @@ -539,14 +534,12 @@ int handleSeek(FILE * fp, unsigned int * permission, int argArrayLength, song = strtol(argArray[1],&test,10); if(*test!='\0') { - myfprintf(fp,"%s \"%s\" is not a integer\n", - COMMAND_RESPOND_ERROR,argArray[1]); + commandError(fp, "\"%s\" is not a integer", argArray[1]); return -1; } time = strtol(argArray[2],&test,10); if(*test!='\0') { - myfprintf(fp,"%s \"%s\" is not a integer\n", - COMMAND_RESPOND_ERROR,argArray[2]); + commandError(fp, "\"%s\" is not a integer", argArray[2]); return -1; } return seekSongInPlaylist(fp,song,time); @@ -571,7 +564,7 @@ int handlePassword(FILE * fp, unsigned int * permission, int argArrayLength, char ** argArray) { if(getPermissionFromPassword(argArray[1],permission)<0) { - myfprintf(fp,"%s incorrect password\n",COMMAND_RESPOND_ERROR); + commandError(fp, "incorrect password"); return -1; } @@ -586,8 +579,7 @@ int handleCrossfade(FILE * fp, unsigned int * permission, int argArrayLength, time = strtol(argArray[1],&test,10); if(*test!='\0' || time<0) { - myfprintf(fp,"%s \"%s\" is not a integer >= 0\n", - COMMAND_RESPOND_ERROR,argArray[1]); + commandError(fp, "\"%s\" is not a integer >= 0", argArray[1]); return -1; } @@ -652,8 +644,7 @@ int checkArgcAndPermission(CommandEntry * cmd, FILE *fp, if (cmd->reqPermission != (permission & cmd->reqPermission)) { if(fp) { - myfprintf(fp,"%s You don't have permission for " - "\"%s\"\n",COMMAND_RESPOND_ERROR, + commandError(fp, "you don't have permission for \"%s\"", cmd->cmd); } return -1; @@ -663,23 +654,22 @@ int checkArgcAndPermission(CommandEntry * cmd, FILE *fp, if (min == max && max != argc) { if(fp) { - myfprintf(fp,"%s Wrong number of arguments for " - "\"%s\"\n",COMMAND_RESPOND_ERROR, + commandError(fp, "wrong number of arguments for \"%s\"", argArray[0]); } return -1; } else if (argc < min) { if(fp) { - myfprintf(fp,"%s too few arguments for \"%s\"\n", - COMMAND_RESPOND_ERROR,argArray[0]); + commandError(fp, "too few arguments for \"%s\"", + argArray[0]); } return -1; } else if (argc > max && max /* != 0 */) { if(fp) { - myfprintf(fp,"%s too many arguments for \"%s\"\n", - COMMAND_RESPOND_ERROR,argArray[0]); + commandError(fp, "too many arguments for \"%s\"", + argArray[0]); } return -1; } @@ -694,10 +684,7 @@ CommandEntry * getCommandEntryAndCheckArgcAndPermission(FILE * fp, if(argArrayLength == 0) return NULL; if(!findInList(commandList, argArray[0],(void *)&cmd)) { - if(fp) { - myfprintf(fp,"%s Unknown command \"%s\"\n", - COMMAND_RESPOND_ERROR,argArray[0]); - } + if(fp) commandError(fp, "unknown command \"%s\"", argArray[0]); return NULL; } diff --git a/src/command.h b/src/command.h index a2733ed6e..3aad04dca 100644 --- a/src/command.h +++ b/src/command.h @@ -22,15 +22,13 @@ #include "../config.h" #include "list.h" +#include "myfprintf.h" #include #define COMMAND_RETURN_KILL 10 #define COMMAND_RETURN_CLOSE 20 -#define COMMAND_RESPOND_ERROR "ACK" -#define COMMAND_RESPOND_OK "OK" - int proccessListOfCommands(FILE * fp, int * permission, int * expired, List * list); @@ -40,5 +38,8 @@ void initCommands(); void finishCommands(); +#define commandSuccess(fp) myfprintf(fp, "OK\n") + +#define commandError(fp, format, ... ) myfprintf(fp, "ACK " format "\n", ##__VA_ARGS__) + #endif -/* vim:set shiftwidth=4 tabstop=8 expandtab: */ diff --git a/src/directory.c b/src/directory.c index 79b3b1bb9..f73a4f14c 100644 --- a/src/directory.c +++ b/src/directory.c @@ -166,7 +166,7 @@ void readDirectoryDBIfUpdateIsFinished() { int updateInit(FILE * fp, List * pathList) { if(directory_updatePid > 0) { - myfprintf(fp,"%s already updating\n",COMMAND_RESPOND_ERROR); + commandError(fp, "already updating"); return -1; } @@ -223,8 +223,7 @@ int updateInit(FILE * fp, List * pathList) { else if(directory_updatePid < 0) { unblockSignals(); ERROR("updateInit: Problems forking()'ing\n"); - myfprintf(fp,"%s problems trying to update\n", - COMMAND_RESPOND_ERROR); + commandError(fp, "problems trying to update"); directory_updatePid = 0; return -1; } @@ -745,7 +744,7 @@ int printDirectoryInfo(FILE * fp, char * name) { Directory * directory; if((directory = getDirectory(name))==NULL) { - myfprintf(fp,"%s: directory not found\n",COMMAND_RESPOND_ERROR); + commandError(fp, "directory not found"); return -1; } @@ -988,13 +987,13 @@ int updateMp3Directory(FILE * fp) { break; default: ERROR("problems updating music db\n"); - myfprintf(fp,"%s problems updating music db\n",COMMAND_RESPOND_ERROR); + commandError(fp, "problems updating music db"); return -1; } if(writeDirectoryDB()<0) { ERROR("problems writing music db file, \"%s\"\n",directory_db); - myfprintf(fp,"%s problems writing music db\n",COMMAND_RESPOND_ERROR); + commandError(fp, "problems writing music db"); return -1; } @@ -1048,7 +1047,7 @@ int traverseAllIn(FILE * fp, char * name, if((song = getSongFromDB(name)) && forEachSong) { return forEachSong(fp, song, data); } - myfprintf(fp,"%s: directory or file not found\n",COMMAND_RESPOND_ERROR); + commandError(fp, "directory or file not found"); return -1; } @@ -1130,7 +1129,7 @@ int searchForSongsIn(FILE * fp, char * name, char * item, char * string) { ret = traverseAllIn(fp,name,searchForFilenameInDirectory,NULL, (void *)dup); } - else myfprintf(fp,"%s unknown table\n",COMMAND_RESPOND_ERROR); + else commandError(fp, "unknown table"); free(dup); @@ -1167,7 +1166,7 @@ int findSongsIn(FILE * fp, char * name, char * item, char * string) { (void *)string); } - myfprintf(fp,"%s unknown table\n",COMMAND_RESPOND_ERROR); + commandError(fp, "unknown table"); return -1; } diff --git a/src/interface.c b/src/interface.c index 2fb95ae1c..4abbc8c4f 100644 --- a/src/interface.c +++ b/src/interface.c @@ -41,7 +41,7 @@ #include #include -#define GREETING "MPD" +#define GREETING "OK MPD" #define INTERFACE_MAX_BUFFER_LENGTH MAXPATHLEN+1024 #define INTERFACE_LIST_MODE_BEGIN "command_list_begin" @@ -120,8 +120,7 @@ void openInterface(Interface * interface, int fd) { #endif interface->outBuffer = malloc(interface->outBufSize); - myfprintf(interface->fp,"%s %s %s\n",COMMAND_RESPOND_OK,GREETING, - VERSION); + myfprintf(interface->fp, "%s %s\n", GREETING, VERSION); printInterfaceOutBuffer(interface); } @@ -237,9 +236,7 @@ int interfaceReadInput(Interface * interface) { interface->num, ret); if(ret==0) { - myfprintf(interface->fp, - "%s\n", - COMMAND_RESPOND_OK); + commandSuccess(interface->fp); } else if(ret==COMMAND_RETURN_CLOSE || interface->expired) { @@ -291,7 +288,8 @@ int interfaceReadInput(Interface * interface) { INTERFACE_LIST_MODE_END) ==0) { - myfprintf(interface->fp,"%s not in command list mode\n",COMMAND_RESPOND_ERROR); + commandError(interface->fp, + "not in command list mode"); ret = -1; } else { @@ -304,7 +302,7 @@ int interfaceReadInput(Interface * interface) { DEBUG("interface %i: command returned %i\n",interface->num,ret); } if(ret==0) { - myfprintf(interface->fp,"%s\n",COMMAND_RESPOND_OK); + commandSuccess(interface->fp); } else if(ret==COMMAND_RETURN_CLOSE || interface->expired) { diff --git a/src/player.c b/src/player.c index 4b96f8584..b4b4e8b74 100644 --- a/src/player.c +++ b/src/player.c @@ -386,8 +386,7 @@ int playerSeek(FILE * fp, Song * song, float time) { PlayerControl * pc = &(getPlayerData()->playerControl); if(pc->state==PLAYER_STATE_STOP) { - myfprintf(fp,"%s player not currently playing\n", - COMMAND_RESPOND_ERROR); + commandError(fp, "player not currently playing"); return -1; } diff --git a/src/playlist.c b/src/playlist.c index 75b812338..d1691b5ed 100644 --- a/src/playlist.c +++ b/src/playlist.c @@ -375,7 +375,7 @@ int playlistInfo(FILE * fp,int song) { end = song+1; } if(song>=playlist.length) { - myfprintf(fp,"%s song doesn't exist\n",COMMAND_RESPOND_ERROR); + commandError(fp, "song doesn't exist"); return -1; } @@ -486,9 +486,8 @@ int addToPlaylist(FILE * fp, char * url) { { } else { - myfprintf(fp,"%s \"%s\" is not in the music db or is" - "not a valid url\n", - COMMAND_RESPOND_ERROR,url); + commandError(fp, "\"%s\" is not in the music db or is" + "not a valid url\n", url); return -1; } @@ -497,7 +496,7 @@ int addToPlaylist(FILE * fp, char * url) { int addSongToPlaylist(FILE * fp, Song * song) { if(playlist.length==playlist_max_length) { - myfprintf(fp,"%s playlist is at the max size\n",COMMAND_RESPOND_ERROR); + commandError(fp, "playlist is at the max size"); return -1; } @@ -536,13 +535,11 @@ int swapSongsInPlaylist(FILE * fp, int song1, int song2) { int currentSong = -1; if(song1<0 || song1>=playlist.length) { - myfprintf(fp,"%s \"%i\" is not in the playlist\n", - COMMAND_RESPOND_ERROR,song1); + commandError(fp,"\"%i\" is not in the playlist", song1); return -1; } if(song2<0 || song2>=playlist.length) { - myfprintf(fp,"%s \"%i\" is not in the playlist\n", - COMMAND_RESPOND_ERROR,song2); + commandError(fp, "\"%i\" is not in the playlist", song2); return -1; } @@ -589,11 +586,11 @@ int deleteFromPlaylist(FILE * fp, int song) { int songOrder; if(song<0) { - myfprintf(fp,"%s need a positive integer\n",COMMAND_RESPOND_ERROR); + commandError(fp, "need a positive integer"); return -1; } if(song>=playlist.length) { - myfprintf(fp,"%s song doesn't exist\n",COMMAND_RESPOND_ERROR); + commandError(fp, "song doesn't exist"); return -1; } @@ -717,17 +714,17 @@ int playPlaylist(FILE * fp, int song, int stopOnError) { } } else if(song<0) { - myfprintf(fp,"%s need integer >= -1\n",COMMAND_RESPOND_ERROR); + commandError(fp, "need integer >= -1"); playlist_state = PLAYLIST_STATE_STOP; return -1; } if(!playlist.length) { - myfprintf(fp,"%s playlist is empty\n",COMMAND_RESPOND_ERROR); + commandError(fp, "playlist is empty"); playlist_state = PLAYLIST_STATE_STOP; return -1; } else if(song>=playlist.length) { - myfprintf(fp,"%s song doesn't exist\n",COMMAND_RESPOND_ERROR); + commandError(fp, "song doesn't exist"); playlist_state = PLAYLIST_STATE_STOP; return -1; } @@ -848,7 +845,7 @@ int getPlaylistRandomStatus() { int setPlaylistRepeatStatus(FILE * fp, int status) { if(status!=0 && status!=1) { - myfprintf(fp,"%s \"%i\" is not 0 or 1\n",COMMAND_RESPOND_ERROR,status); + commandError(fp, "\"%i\" is not 0 or 1", status); return -1; } @@ -872,14 +869,12 @@ int moveSongInPlaylist(FILE * fp, int from, int to) { int currentSong = -1; if(from<0 || from>=playlist.length) { - myfprintf(fp,"%s \"%i\" is not a song in the playlist\n", - COMMAND_RESPOND_ERROR,from); + commandError(fp, "\"%i\" is not a song in the playlist", from); return -1; } if(to<0 || to>=playlist.length) { - myfprintf(fp,"%s \"%i\" is not a song in the playlist\n", - COMMAND_RESPOND_ERROR,to); + commandError(fp, "\"%i\" is not a song in the playlist", to); return -1; } @@ -985,7 +980,7 @@ int setPlaylistRandomStatus(FILE * fp, int status) { int statusWas = playlist.random; if(status!=0 && status!=1) { - myfprintf(fp,"%s \"%i\" is not 0 or 1\n",COMMAND_RESPOND_ERROR,status); + commandError(fp, "\"%i\" is not 0 or 1", status); return -1; } @@ -1079,13 +1074,12 @@ int deletePlaylist(FILE * fp, char * utf8file) { if((actualFile = rpp2app(rfile)) && isPlaylist(actualFile)) free(rfile); else { free(rfile); - myfprintf(fp,"%s playlist \"%s\" not found\n", - COMMAND_RESPOND_ERROR,utf8file); + commandError(fp, "playlist \"%s\" not found", utf8file); return -1; } if(unlink(actualFile)<0) { - myfprintf(fp,"%s problems deleting file\n",COMMAND_RESPOND_ERROR); + commandError(fp, "problems deleting file"); return -1; } @@ -1101,9 +1095,8 @@ int savePlaylist(FILE * fp, char * utf8file) { char * actualFile; if(strstr(utf8file,"/")) { - myfprintf(fp,"%s cannot save \"%s\", saving playlists to " - "subdirectories is not supported\n", - COMMAND_RESPOND_ERROR,utf8file); + commandError(fp, "cannot save \"%s\", saving playlists to " + "subdirectories is not supported", utf8file); return -1; } @@ -1123,13 +1116,14 @@ int savePlaylist(FILE * fp, char * utf8file) { free(rfile); if(0==stat(actualFile,&st)) { - myfprintf(fp,"%s A file or directory already exists with the name \"%s\"\n",COMMAND_RESPOND_ERROR,utf8file); + myfprintf(fp, "a file or directory already exists with the name" + " \"%s\"", utf8file); return -1; } while(!(fileP = fopen(actualFile,"w")) && errno==EINTR); if(fileP==NULL) { - myfprintf(fp,"%s Problems opening file\n",COMMAND_RESPOND_ERROR); + commandError(fp, "problems opening file"); return -1; } @@ -1171,15 +1165,13 @@ int loadPlaylist(FILE * fp, char * utf8file) { if((actualFile = rpp2app(rfile)) && isPlaylist(actualFile)) free(rfile); else { free(rfile); - myfprintf(fp,"%s playlist \"%s\" not found\n", - COMMAND_RESPOND_ERROR,utf8file); + commandError(fp, "playlist \"%s\" not found", utf8file); return -1; } while(!(fileP = fopen(actualFile,"r")) && errno==EINTR); if(fileP==NULL) { - myfprintf(fp,"%s Problems opening file \"%s\"\n", - COMMAND_RESPOND_ERROR,utf8file); + commandError(fp, "problems opening file \"%s\"", utf8file); return -1; } @@ -1197,7 +1189,8 @@ int loadPlaylist(FILE * fp, char * utf8file) { strncat(s,"/",MAXPATHLEN-parentlen); strncat(s,temp,MAXPATHLEN-parentlen-1); if(strlen(s)>=MAXPATHLEN) { - myfprintf(fp,"%s \"%s\" too long\n",COMMAND_RESPOND_ERROR,temp); + commandError(fp, "\"%s\" too long", + temp); free(temp); while(fclose(fileP) && errno==EINTR); if(erroredFile) free(erroredFile); @@ -1222,7 +1215,7 @@ int loadPlaylist(FILE * fp, char * utf8file) { } else if(slength==MAXPATHLEN) { s[slength] = '\0'; - myfprintf(fp,"%s \"%s\" too long\n",COMMAND_RESPOND_ERROR,s); + commandError(fp, "\"%s\" too long", s); while(fclose(fileP) && errno==EINTR); if(erroredFile) free(erroredFile); return -1; @@ -1233,8 +1226,7 @@ int loadPlaylist(FILE * fp, char * utf8file) { while(fclose(fileP) && errno==EINTR); if(erroredFile) { - myfprintf(fp,"%s can't add file \"%s\"\n",COMMAND_RESPOND_ERROR, - erroredFile); + commandError(fp, "can't add file \"%s\"", erroredFile); free(erroredFile); return -1; } @@ -1262,15 +1254,15 @@ int seekSongInPlaylist(FILE * fp, int song, float time) { int i = song; if(song<0) { - myfprintf(fp,"%s need integer >= -1\n",COMMAND_RESPOND_ERROR); + commandError(fp, "need integer >= -1"); return -1; } if(!playlist.length) { - myfprintf(fp,"%s playlist is empty\n",COMMAND_RESPOND_ERROR); + commandError(fp, "playlist is empty"); return -1; } else if(song>=playlist.length) { - myfprintf(fp,"%s song doesn't exist\n",COMMAND_RESPOND_ERROR); + commandError(fp, "song doesn't exist"); return -1; } diff --git a/src/tables.c b/src/tables.c index 4129c1d29..114f71d15 100644 --- a/src/tables.c +++ b/src/tables.c @@ -172,8 +172,7 @@ int printAllAlbums(FILE * fp, char * artist) { } } else { - myfprintf(fp,"%s artist \"%s\" not found\n", - COMMAND_RESPOND_ERROR,artist); + commandError(fp, "artist \"%s\" not found", artist); return -1; } } @@ -184,7 +183,7 @@ int printAllAlbums(FILE * fp, char * artist) { int printAllKeysOfTable(FILE * fp, char * table, char * arg1) { if(strcmp(table,TABLES_ARTIST)==0) { if(arg1!=NULL) { - myfprintf(fp,"%s %s table takes no args\n",COMMAND_RESPOND_ERROR,table); + commandError(fp, "%s table takes no args", table); return -1; } return printAllArtists(fp); @@ -193,8 +192,8 @@ int printAllKeysOfTable(FILE * fp, char * table, char * arg1) { return printAllAlbums(fp,arg1); } else { - myfprintf(fp,"%s table \"%s\" does not exist or not available for listing\n",COMMAND_RESPOND_ERROR,table); + commandError(fp, "table \"%s\" does not exist or not available " + "for listing", table); return -1; } } -/* vim:set shiftwidth=4 tabstop=8 expandtab: */ diff --git a/src/volume.c b/src/volume.c index 49d7483cd..bf367ab75 100644 --- a/src/volume.c +++ b/src/volume.c @@ -144,8 +144,7 @@ int changeOssVolumeLevel(FILE * fp, int change, int rel) { if (rel) { if((current = getOssVolumeLevel()) < 0) { - myfprintf(fp,"%s problem getting current volume\n", - COMMAND_RESPOND_ERROR); + commandError(fp, "problem getting current volume"); return -1; } @@ -159,7 +158,7 @@ int changeOssVolumeLevel(FILE * fp, int change, int rel) { level = (new << 8) + new; if(ioctl(volume_ossFd,MIXER_WRITE(volume_ossControl),&level) < 0) { - myfprintf(fp,"%s problems setting volume\n",COMMAND_RESPOND_ERROR); + commandError(fp, "problems setting volume"); return -1; } @@ -271,8 +270,7 @@ int changeAlsaVolumeLevel(FILE * fp, int change, int rel) { if((err = snd_mixer_selem_get_playback_volume(volume_alsaElem, SND_MIXER_SCHN_FRONT_LEFT,&level))<0) { - myfprintf(fp,"%s problems getting volume\n", - COMMAND_RESPOND_ERROR); + commandError(fp, "problems getting volume"); ERROR("problems getting alsa volume: %s\n",snd_strerror(err)); return -1; } @@ -298,8 +296,7 @@ int changeAlsaVolumeLevel(FILE * fp, int change, int rel) { if((err = snd_mixer_selem_set_playback_volume_all( volume_alsaElem,level))<0) { - myfprintf(fp,"%s problems setting volume\n", - COMMAND_RESPOND_ERROR); + commandError(fp, "problems setting volume"); ERROR("problems setting alsa volume: %s\n",snd_strerror(err)); return -1; } @@ -426,7 +423,7 @@ int changeVolumeLevel(FILE * fp, int change, int rel) { case VOLUME_MIXER_TYPE_SOFTWARE: return changeSoftwareVolume(fp,change,rel); default: - myfprintf(fp,"%s no volume support!\n",COMMAND_RESPOND_ERROR); + commandError(fp, "no volume support"); return -1; } } -- cgit v1.2.3