From: aidan Date: Fri, 14 Aug 2020 18:36:38 +0000 (-0600) Subject: Manual merge of pull request #19. This adds multi-character delimiter. X-Git-Url: https://git.danieliu.xyz/?a=commitdiff_plain;h=941f415b3821bf4f3493a0864c05a1af313b22bc;p=dwmblocks.git Manual merge of pull request #19. This adds multi-character delimiter. Thanks to tph5595. --- diff --git a/blocks.def.h b/blocks.def.h index 35738db..9c22d68 100644 --- a/blocks.def.h +++ b/blocks.def.h @@ -7,4 +7,5 @@ static const Block blocks[] = { }; //sets delimeter between status commands. NULL character ('\0') means no delimeter. -static char delim = '|'; +static char delim[] = " | "; +static unsigned int delimLen = 5; diff --git a/dwmblocks.c b/dwmblocks.c index 0de8630..e3ac034 100644 --- a/dwmblocks.c +++ b/dwmblocks.c @@ -13,6 +13,7 @@ #endif #define LENGTH(X) (sizeof(X) / sizeof (X[0])) #define CMDLENGTH 50 +#define MIN( a, b ) ( ( a < b) ? a : b ) #define STATUSLENGTH (LENGTH(blocks) * CMDLENGTH + 1) typedef struct { @@ -26,13 +27,13 @@ void dummysighandler(int num); #endif void sighandler(int num); void getcmds(int time); -void getsigcmds(int signal); +void getsigcmds(unsigned int signal); void setupsignals(); void sighandler(int signum); int getstatus(char *str, char *last); void setroot(); void statusloop(); -void termhandler(int signum); +void termhandler(); #include "blocks.h" @@ -55,18 +56,19 @@ void getcmd(const Block *block, char *output) return; char c; int i = strlen(block->icon); - fgets(output+i, CMDLENGTH-i, cmdf); + fgets(output+i, CMDLENGTH-i-delimLen, cmdf); i = strlen(output); - if (delim != '\0' && --i) - output[i++] = delim; - output[i++] = '\0'; + if (delim[0] != '\0' && --i) + strncpy(output+i, delim, delimLen); + else + output[i++] = '\0'; pclose(cmdf); } void getcmds(int time) { const Block* current; - for(int i = 0; i < LENGTH(blocks); i++) + for(unsigned int i = 0; i < LENGTH(blocks); i++) { current = blocks + i; if ((current->interval != 0 && time % current->interval == 0) || time == -1) @@ -74,10 +76,10 @@ void getcmds(int time) } } -void getsigcmds(int signal) +void getsigcmds(unsigned int signal) { const Block *current; - for (int i = 0; i < LENGTH(blocks); i++) + for (unsigned int i = 0; i < LENGTH(blocks); i++) { current = blocks + i; if (current->signal == signal) @@ -93,7 +95,7 @@ void setupsignals() signal(i, dummysighandler); #endif - for(int i = 0; i < LENGTH(blocks); i++) + for(unsigned int i = 0; i < LENGTH(blocks); i++) { if (blocks[i].signal > 0) signal(SIGMINUS+blocks[i].signal, sighandler); @@ -105,9 +107,9 @@ int getstatus(char *str, char *last) { strcpy(last, str); str[0] = '\0'; - for(int i = 0; i < LENGTH(blocks); i++) + for(unsigned int i = 0; i < LENGTH(blocks); i++) strcat(str, statusbar[i]); - str[strlen(str)-1] = '\0'; + str[strlen(str)-strlen(delim)] = '\0'; return strcmp(str, last);//0 if they are the same } @@ -162,7 +164,7 @@ void sighandler(int signum) writestatus(); } -void termhandler(int signum) +void termhandler() { statusContinue = 0; } @@ -172,10 +174,11 @@ int main(int argc, char** argv) for(int i = 0; i < argc; i++) { if (!strcmp("-d",argv[i])) - delim = argv[++i][0]; + strncpy(delim, argv[++i], delimLen); else if(!strcmp("-p",argv[i])) writestatus = pstdout; } + delim[MIN(delimLen, strlen(delim))] = '\0'; signal(SIGTERM, termhandler); signal(SIGINT, termhandler); statusloop();