#define DEBUG_FILE "debug.txt"
-#define MAX_ATTRIBUTE_CHARS (5 * 2)
+#define MAX_ATTRIBUTE_CHARS (6 * 2)
-#define VERSION "0.60"
+#define VERSION "0.62"
typedef enum { false, true } bool;
{ NULL, 0, NULL, 0 },
};
+enum attr_type {
+ ATTR_BOLD = 0x01,
+ ATTR_UNDERSCORE = 0x02,
+ ATTR_BLINK = 0x04,
+ ATTR_REVERSE = 0x08,
+ ATTR_CONCEALED = 0x10
+};
+struct attr {
+ const char *name;
+ unsigned int val;
+ enum attr_type type;
+};
+
static FILE *stream;
#if DEBUG
static FILE *log;
static void process_opts (int, char **);
static void process_opt_attr (const char *);
-static void write_attr (unsigned int);
+static void write_attr (const struct attr *, unsigned int *);
static void print_hint (void);
static void print_help (void);
static void print_version (void);
case 0: /* long opts */
switch (opt_type)
{
- case OPT_ATTR: {
- char *opt;
- opt = xstrdup (optarg);
- process_opt_attr (opt);
- free (opt);
+ case OPT_ATTR:
+ process_opt_attr (optarg);
break;
- }
case OPT_CLEAN:
clean = true;
break;
static void
process_opt_attr (const char *p)
{
+ /* If attributes are added to this "list", also increase MAX_ATTRIBUTE_CHARS! */
+ const struct attr attrs[] = {
+ { "bold", 1, ATTR_BOLD },
+ { "underscore", 4, ATTR_UNDERSCORE },
+ { "blink", 5, ATTR_BLINK },
+ { "reverse", 7, ATTR_REVERSE },
+ { "concealed", 8, ATTR_CONCEALED },
+ };
+ unsigned int attr_types = 0;
+
while (*p)
{
const char *s;
vfprintf_fail (formats[FMT_GENERIC], "--attr switch must have strings separated by ,");
else
{
- /* If atttributes are added to this "list", also increase MAX_ATTRIBUTE_CHARS! */
- if (p - s == 4 && strneq (s, "bold", 4))
- write_attr (1);
- else if (p - s == 10 && strneq (s, "underscore", 10))
- write_attr (4);
- else if (p - s == 5 && strneq (s, "blink", 5))
- write_attr (5);
- else if (p - s == 7 && strneq (s, "reverse", 7))
- write_attr (7);
- else if (p - s == 9 && strneq (s, "concealed", 9))
- write_attr (8);
- else
- vfprintf_fail (formats[FMT_GENERIC], "--attr switch must be provided valid attribute names");
+ bool valid_attr = false;
+ unsigned int i;
+ for (i = 0; i < sizeof (attrs) / sizeof (struct attr); i++)
+ {
+ const size_t name_len = strlen (attrs[i].name);
+ if ((size_t)(p - s) == name_len && strneq (s, attrs[i].name, name_len))
+ {
+ write_attr (&attrs[i], &attr_types);
+ valid_attr = true;
+ break;
+ }
+ }
+ if (!valid_attr)
+ {
+ char *attr_invalid = xmalloc ((p - s) + 1);
+ STACK_VAR (attr_invalid);
+ strncpy (attr_invalid, s, p - s);
+ attr_invalid[p - s] = '\0';
+ vfprintf_fail ("--attr switch attribute '%s' is not valid", attr_invalid);
+ RELEASE_VAR (attr_invalid); /* never reached */
+ }
}
if (*p)
p++;
}
static void
-write_attr (unsigned int val)
+write_attr (const struct attr *attr_i, unsigned int *attr_types)
{
+ const unsigned int val = attr_i->val;
+ const enum attr_type attr_type = attr_i->type;
+ const char *attr_name = attr_i->name;
+
+ if (*attr_types & attr_type)
+ vfprintf_fail ("--attr switch has attribute '%s' twice or more", attr_name);
snprintf (attr + strlen (attr), 3, "%u;", val);
+ *attr_types |= attr_type;
}
static void
const char *short_opt = NULL;
unsigned int i;
for (i = 0; i < sizeof (short_opts) / sizeof (struct short_opt); i++)
- {
- if (streq (opt->name, short_opts[i].name))
- {
- short_opt = short_opts[i].short_opt;
- break;
- }
- }
+ if (streq (opt->name, short_opts[i].name))
+ {
+ short_opt = short_opts[i].short_opt;
+ break;
+ }
if (short_opt)
printf ("\t\t-%s, --%s\n", short_opt, opt->name);
else
#endif
const char *version_prefix, *version_string;
const char *c_flags, *ld_flags, *cpp_flags;
+ const char *const desc_flags_unknown = "unknown";
struct bytes_size bytes_size;
bool debug;
#ifdef CFLAGS
c_flags = to_str (CFLAGS);
#else
- c_flags = "unknown";
+ c_flags = desc_flags_unknown;
#endif
#ifdef LDFLAGS
ld_flags = to_str (LDFLAGS);
#else
- ld_flags = "unknown";
+ ld_flags = desc_flags_unknown;
#endif
#ifdef CPPFLAGS
cpp_flags = to_str (CPPFLAGS);
#else
- cpp_flags = "unknown";
+ cpp_flags = desc_flags_unknown;
#endif
#if DEBUG
debug = true;
const char *color_string = arg_cnt >= 1 ? arg_strings[0] : NULL;
const char *file_string = arg_cnt == 2 ? arg_strings[1] : NULL;
- assert (color_string);
+ assert (color_string != NULL);
if (streq (color_string, "-"))
{
gather_color_names (color_string, attr, color_names);
- assert (color_names[FOREGROUND]);
+ assert (color_names[FOREGROUND] != NULL);
if (color_names[BACKGROUND])
{
}
find_color_entries (color_names, colors);
+ assert (colors[FOREGROUND] != NULL);
free_color_names (color_names);
if (!colors[FOREGROUND]->code && colors[BACKGROUND] && colors[BACKGROUND]->code)
*file = "stdin";
}
- assert (*stream);
- assert (*file);
+ assert (*stream != NULL);
+ assert (*file != NULL);
}
static void
if (have_file)
{
- const char *file_exists = color_string;
+ const char *file_existing = color_string;
if (file_string)
- vfprintf_fail (formats[FMT_QUOTE], get_file_type (mode), file_exists, "cannot be used as color string");
+ vfprintf_fail (formats[FMT_QUOTE], get_file_type (mode), file_existing, "cannot be used as color string");
else
{
if (VALID_FILE_TYPE (mode))
- vfprintf_fail (formats[FMT_QUOTE], get_file_type (mode), file_exists, "must be preceded by color string");
+ vfprintf_fail (formats[FMT_QUOTE], get_file_type (mode), file_existing, "must be preceded by color string");
else
- vfprintf_fail (formats[FMT_QUOTE], get_file_type (mode), file_exists, "is not a valid file type");
+ vfprintf_fail (formats[FMT_QUOTE], get_file_type (mode), file_existing, "is not a valid file type");
}
}
}
}
else
p = color + strlen (color);
- assert (p);
+ assert (p != NULL);
for (ch = color; *ch; ch++)
if (!isalpha (*ch))
if (read_from_stream)
save_char (ch, buf, &i, &size);
}
- else /* read next character */
+ else /* got next character */
{
got_next_char = true;
break;
projects / colorize.git / blobdiff