alex/swayfx
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

bar_cmd_colors: remove add_color

Browse source 
This is the third commit in a series of commits to refactor color
handling in sway. This removes add_color from commands.c. It was only
being used by bar_cmd_colors. This also changes the functions to use
parse_color which is used to validate rgb(a) colors throughout the code
base and is also what i3bar is using to parse the colors after they are
passed over ipc. After parsing the color and ensuring it is valid, the
rgba hex string is then generated using snprintf. This refactor also
ensures that all the colors for the command are valid before applying
any of them.
This commit is contained in:
Brian Ashworth 2019-12-28 04:01:38 -05:00 committed by Simon Ser
parent 66dc33296c
commit f898ca9a83
3 changed files with 33 additions and 49 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
include/sway/commands.h
View file

@ -88,8 +88,6 @@ void free_cmd_results(struct cmd_results *results);
*/
char *cmd_results_to_json(list_t *res_list);
struct cmd_results *add_color(char *buffer, const char *color);
/**
* TODO: Move this function and its dependent functions to container.c.
*/

31
sway/commands.c
View file

@ -567,34 +567,3 @@ char *cmd_results_to_json(list_t *res_list) {
json_object_put(result_array);
return res;
}
/**
* Check and add color to buffer.
*
* return error object, or NULL if color is valid.
*/
struct cmd_results *add_color(char *buffer, const char *color) {
int len = strlen(color);
if (len != 7 && len != 9) {
return cmd_results_new(CMD_INVALID,
"Invalid color definition %s", color);
}
if (color[0] != '#') {
return cmd_results_new(CMD_INVALID,
"Invalid color definition %s", color);
}
for (int i = 1; i < len; ++i) {
if (!isxdigit(color[i])) {
return cmd_results_new(CMD_INVALID,
"Invalid color definition %s", color);
}
}
strcpy(buffer, color);
// add default alpha channel if color was defined without it
if (len == 7) {
buffer[7] = 'f';
buffer[8] = 'f';
}
buffer[9] = '\0';
return NULL;
}

49
sway/commands/bar/colors.c
View file

@ -1,5 +1,7 @@
#include <string.h>
#include "sway/commands.h"
#include "log.h"
#include "util.h"
// Must be in alphabetical order for bsearch
static struct cmd_handler bar_colors_handlers[] = {
@ -16,38 +18,53 @@ static struct cmd_handler bar_colors_handlers[] = {
{ "urgent_workspace", bar_colors_cmd_urgent_workspace },
};
static char *hex_to_rgba_hex(const char *hex) {
uint32_t color;
if (!parse_color(hex, &color)) {
return NULL;
}
char *rgba = malloc(10);
snprintf(rgba, 10, "#%08x", color);
return rgba;
}
static struct cmd_results *parse_single_color(char **color,
const char *cmd_name, int argc, char **argv) {
struct cmd_results *error = NULL;
if ((error = checkarg(argc, cmd_name, EXPECTED_EQUAL_TO, 1))) {
return error;
}
if (!*color && !(*color = malloc(10))) {
return NULL;
}
error = add_color(*color, argv[0]);
if (error) {
return error;
char *rgba = hex_to_rgba_hex(argv[0]);
if (!*rgba) {
return cmd_results_new(CMD_INVALID, "Invalid color: %s", argv[0]);
}
free(*color);
*color = rgba;
return cmd_results_new(CMD_SUCCESS, NULL);
}
static struct cmd_results *parse_three_colors(char ***colors,
const char *cmd_name, int argc, char **argv) {
struct cmd_results *error = NULL;
if (argc != 3) {
return cmd_results_new(CMD_INVALID,
"Command '%s' requires exactly three color values", cmd_name);
if ((error = checkarg(argc, cmd_name, EXPECTED_EQUAL_TO, 3))) {
return error;
}
for (size_t i = 0; i < 3; i++) {
if (!*colors[i] && !(*(colors[i]) = malloc(10))) {
return NULL;
}
error = add_color(*(colors[i]), argv[i]);
if (error) {
return error;
char *rgba[3] = {0};
for (int i = 0; i < 3; i++) {
rgba[i] = hex_to_rgba_hex(argv[i]);
if (!rgba[i]) {
return cmd_results_new(CMD_INVALID, "Invalid color: %s", argv[i]);
}
}
for (int i = 0; i < 3; i++) {
free(*colors[i]);
*colors[i] = rgba[i];
}
return cmd_results_new(CMD_SUCCESS, NULL);
}