X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=src%2Fdata%2Fformat.c;h=8aa16e65fffc63f045a4d60e205a59dd9bb2536b;hb=4daafd79f8b250b1651c1b66a3171a654abf252d;hp=e15931ad1a2b4764547747945e9399305f2b8946;hpb=fe8dc2171009e90d2335f159d05f7e6660e24780;p=pspp diff --git a/src/data/format.c b/src/data/format.c index e15931ad1a..8aa16e65ff 100644 --- a/src/data/format.c +++ b/src/data/format.c @@ -1,5 +1,5 @@ /* PSPP - a program for statistical analysis. - Copyright (C) 1997-9, 2000, 2006, 2010, 2011 Free Software Foundation, Inc. + Copyright (C) 1997-9, 2000, 2006, 2010, 2011, 2012 Free Software Foundation, Inc. This program is free software: you can redistribute it and/or modify it under the terms of the GNU General Public License as published by @@ -33,73 +33,55 @@ #include "libpspp/misc.h" #include "libpspp/str.h" +#include "gl/c-strcase.h" #include "gl/minmax.h" #include "gl/xalloc.h" +#include "gl/xmemdup0.h" #include "gettext.h" #define _(msgid) gettext (msgid) -struct fmt_settings - { - struct fmt_number_style styles[FMT_NUMBER_OF_FORMATS]; - }; - bool is_fmt_type (enum fmt_type); -static bool valid_width (enum fmt_type, int width, bool for_input); +static bool valid_width (enum fmt_type, int width, enum fmt_use); static int max_digits_for_bytes (int bytes); +static void fmt_clamp_width (struct fmt_spec *, enum fmt_use); +static void fmt_clamp_decimals (struct fmt_spec *, enum fmt_use); -static void fmt_affix_set (struct fmt_affix *, const char *); -static void fmt_affix_free (struct fmt_affix *); - -static void fmt_number_style_init (struct fmt_number_style *); -static void fmt_number_style_clone (struct fmt_number_style *, - const struct fmt_number_style *); -static void fmt_number_style_destroy (struct fmt_number_style *); - -/* Creates and returns a new struct fmt_settings with default format styles. */ -struct fmt_settings * -fmt_settings_create (void) +void +fmt_settings_init (struct fmt_settings *settings) { - struct fmt_settings *settings; - int t; - - settings = xzalloc (sizeof *settings); - for (t = 0 ; t < FMT_NUMBER_OF_FORMATS ; ++t ) - fmt_number_style_init (&settings->styles[t]); - fmt_settings_set_decimal (settings, '.'); - - return settings; + *settings = (struct fmt_settings) FMT_SETTINGS_INIT; } -/* Destroys SETTINGS. */ void -fmt_settings_destroy (struct fmt_settings *settings) +fmt_settings_uninit (struct fmt_settings *settings) { - if (settings != NULL) - { - int t; - - for (t = 0 ; t < FMT_NUMBER_OF_FORMATS ; ++t ) - fmt_number_style_destroy (&settings->styles[t]); - - free (settings->styles); - } + for (int i = 0; i < FMT_N_CCS; i++) + fmt_number_style_destroy (settings->ccs[i]); } -/* Returns a copy of SETTINGS. */ -struct fmt_settings * -fmt_settings_clone (const struct fmt_settings *old) +void +fmt_settings_copy (struct fmt_settings *new, const struct fmt_settings *old) { - struct fmt_settings *new; - int t; - - new = xmalloc (sizeof *new); - for (t = 0 ; t < FMT_NUMBER_OF_FORMATS ; ++t ) - fmt_number_style_clone (&new->styles[t], &old->styles[t]); + new->decimal = old->decimal; + for (int i = 0; i < FMT_N_CCS; i++) + new->ccs[i] = fmt_number_style_clone (old->ccs[i]); +} - return new; +static size_t +fmt_type_to_cc_index (enum fmt_type type) +{ + switch (type) + { + case FMT_CCA: return 0; + case FMT_CCB: return 1; + case FMT_CCC: return 2; + case FMT_CCD: return 3; + case FMT_CCE: return 4; + default: NOT_REACHED (); + } } /* Returns the number formatting style associated with the given @@ -108,59 +90,74 @@ const struct fmt_number_style * fmt_settings_get_style (const struct fmt_settings *settings, enum fmt_type type) { - assert (is_fmt_type (type)); - return &settings->styles[type]; -} + verify (FMT_F < 6); + verify (FMT_COMMA < 6); + verify (FMT_DOT < 6); + verify (FMT_DOLLAR < 6); + verify (FMT_PCT < 6); + verify (FMT_E < 6); + +#define OPPOSITE(C) ((C) == ',' ? '.' : ',') +#define AFFIX(S) { .s = (char *) (S), .width = sizeof (S) - 1 } +#define NS(PREFIX, SUFFIX, DECIMAL, GROUPING) { \ + .neg_prefix = AFFIX ("-"), \ + .prefix = AFFIX (PREFIX), \ + .suffix = AFFIX (SUFFIX), \ + .neg_suffix = AFFIX (""), \ + .decimal = DECIMAL, \ + .grouping = GROUPING, \ + } +#define ANS(DECIMAL, GROUPING) { \ + [FMT_F] = NS( "", "", DECIMAL, 0), \ + [FMT_E] = NS( "", "", DECIMAL, 0), \ + [FMT_COMMA] = NS( "", "", DECIMAL, GROUPING), \ + [FMT_DOT] = NS( "", "", GROUPING, DECIMAL), \ + [FMT_DOLLAR] = NS("$", "", DECIMAL, GROUPING), \ + [FMT_PCT] = NS( "", "%", DECIMAL, 0), \ + } + + static const struct fmt_number_style period_styles[6] = ANS ('.', ','); + static const struct fmt_number_style comma_styles[6] = ANS (',', '.'); + static const struct fmt_number_style default_style = NS ("", "", '.', 0); -/* Sets the number style for TYPE to have the given DECIMAL and GROUPING - characters, negative prefix NEG_PREFIX, prefix PREFIX, suffix SUFFIX, and - negative suffix NEG_SUFFIX. All of the strings are UTF-8 encoded. */ -void -fmt_settings_set_style (struct fmt_settings *settings, enum fmt_type type, - char decimal, char grouping, - const char *neg_prefix, const char *prefix, - const char *suffix, const char *neg_suffix) -{ - struct fmt_number_style *style = &settings->styles[type]; - int total_bytes, total_width; - - assert (grouping == '.' || grouping == ',' || grouping == 0); - assert (decimal == '.' || decimal == ','); - assert (decimal != grouping); - - fmt_number_style_destroy (style); - - fmt_affix_set (&style->neg_prefix, neg_prefix); - fmt_affix_set (&style->prefix, prefix); - fmt_affix_set (&style->suffix, suffix); - fmt_affix_set (&style->neg_suffix, neg_suffix); - style->decimal = decimal; - style->grouping = grouping; - - total_bytes = (strlen (neg_prefix) + strlen (prefix) - + strlen (suffix) + strlen (neg_suffix)); - total_width = (style->neg_prefix.width + style->prefix.width - + style->suffix.width + style->neg_suffix.width); - style->extra_bytes = MAX (0, total_bytes - total_width); -} + switch (type) + { + case FMT_F: + case FMT_COMMA: + case FMT_DOT: + case FMT_DOLLAR: + case FMT_PCT: + case FMT_E: + return (settings->decimal == '.' + ? &period_styles[type] + : &comma_styles[type]); -/* Sets the decimal point character for the settings in S to DECIMAL. + case FMT_CCA: + case FMT_CCB: + case FMT_CCC: + case FMT_CCD: + case FMT_CCE: + { + size_t idx = fmt_type_to_cc_index (type); + return settings->ccs[idx] ? settings->ccs[idx] : &default_style; + } + + default: + return &default_style; + } +} - This has no effect on custom currency formats. */ void -fmt_settings_set_decimal (struct fmt_settings *s, char decimal) +fmt_settings_set_cc (struct fmt_settings *settings, enum fmt_type type, + struct fmt_number_style *style) { - int grouping = decimal == '.' ? ',' : '.'; - assert (decimal == '.' || decimal == ','); - - fmt_settings_set_style (s, FMT_F, decimal, 0, "-", "", "", ""); - fmt_settings_set_style (s, FMT_E, decimal, 0, "-", "", "", ""); - fmt_settings_set_style (s, FMT_COMMA, decimal, grouping, "-", "", "", ""); - fmt_settings_set_style (s, FMT_DOT, grouping, decimal, "-", "", "", ""); - fmt_settings_set_style (s, FMT_DOLLAR, decimal, grouping, "-", "$", "", ""); - fmt_settings_set_style (s, FMT_PCT, decimal, 0, "-", "", "%", ""); + size_t idx = fmt_type_to_cc_index (type); + + fmt_number_style_destroy (settings->ccs[idx]); + settings->ccs[idx] = style; } + /* Returns an input format specification with type TYPE, width W, and D decimals. */ struct fmt_spec @@ -290,6 +287,16 @@ fmt_for_output_from_input (const struct fmt_spec *input) case FMT_MONTH: break; + case FMT_MTIME: + if (input->d) + output.w = MAX (input->w, input->d + 6); + break; + + case FMT_YMDHMS: + if (input->w) + output.w = MAX (input->w, input->d + 20); + break; + default: NOT_REACHED (); } @@ -311,20 +318,20 @@ fmt_default_for_width (int width) : fmt_for_output (FMT_A, width, 0)); } -/* Checks whether SPEC is valid as an input format (if FOR_INPUT) - or an output format (otherwise) and returns nonzero if so. +/* Checks whether SPEC is valid for USE and returns nonzero if so. Otherwise, emits an error message and returns zero. */ bool -fmt_check (const struct fmt_spec *spec, bool for_input) +fmt_check (const struct fmt_spec *spec, enum fmt_use use) { - const char *io_fmt = for_input ? _("Input format") : _("Output format"); + const char *io_fmt; char str[FMT_STRING_LEN_MAX + 1]; int min_w, max_w, max_d; assert (is_fmt_type (spec->type)); fmt_to_string (spec, str); - if (for_input && !fmt_usable_for_input (spec->type)) + io_fmt = use == FMT_FOR_INPUT ? _("Input format") : _("Output format"); + if (use == FMT_FOR_INPUT && !fmt_usable_for_input (spec->type)) { msg (SE, _("Format %s may not be used for input."), str); return false; @@ -338,8 +345,8 @@ fmt_check (const struct fmt_spec *spec, bool for_input) return false; } - min_w = fmt_min_width (spec->type, for_input); - max_w = fmt_max_width (spec->type, for_input); + min_w = fmt_min_width (spec->type, use); + max_w = fmt_max_width (spec->type, use); if (spec->w < min_w || spec->w > max_w) { msg (SE, _("%s %s specifies width %d, but " @@ -348,7 +355,7 @@ fmt_check (const struct fmt_spec *spec, bool for_input) return false; } - max_d = fmt_max_decimals (spec->type, spec->w, for_input); + max_d = fmt_max_decimals (spec->type, spec->w, use); if (!fmt_takes_decimals (spec->type) && spec->d != 0) { msg (SE, ngettext ("%s %s specifies %d decimal place, but " @@ -387,7 +394,7 @@ fmt_check (const struct fmt_spec *spec, bool for_input) bool fmt_check_input (const struct fmt_spec *spec) { - return fmt_check (spec, true); + return fmt_check (spec, FMT_FOR_INPUT); } /* Checks whether SPEC is valid as an output format and returns @@ -395,7 +402,7 @@ fmt_check_input (const struct fmt_spec *spec) bool fmt_check_output (const struct fmt_spec *spec) { - return fmt_check (spec, false); + return fmt_check (spec, FMT_FOR_OUTPUT); } /* Checks that FORMAT is appropriate for a variable of the given @@ -472,8 +479,10 @@ fmt_equal (const struct fmt_spec *a, const struct fmt_spec *b) return a->type == b->type && a->w == b->w && a->d == b->d; } -/* Adjusts FMT to be valid for a value of the given WIDTH. */ -void +/* Adjusts FMT to be valid for a value of the given WIDTH if necessary. + If nothing needed to be changed the return value is false + */ +bool fmt_resize (struct fmt_spec *fmt, int width) { if ((width > 0) != fmt_is_string (fmt->type)) @@ -491,46 +500,31 @@ fmt_resize (struct fmt_spec *fmt, int width) else { /* Still numeric. */ + return false; } + return true; } -/* Adjusts FMT's width and decimal places to be valid for an - input format (if FOR_INPUT) or an output format (if - !FOR_INPUT). */ +/* Adjusts FMT's width and decimal places to be valid for USE. */ void -fmt_fix (struct fmt_spec *fmt, bool for_input) +fmt_fix (struct fmt_spec *fmt, enum fmt_use use) { - int min_w, max_w; - int max_d; - /* Clamp width to those allowed by format. */ - min_w = fmt_min_width (fmt->type, for_input); - max_w = fmt_max_width (fmt->type, for_input); - if (fmt->w < min_w) - fmt->w = min_w; - else if (fmt->w > max_w) - fmt->w = max_w; + fmt_clamp_width (fmt, use); - /* First, if FMT has more decimal places than allowed, attempt - to increase FMT's width until that number of decimal places - can be achieved. */ - if (fmt->d > fmt_max_decimals (fmt->type, fmt->w, for_input)) + /* If FMT has more decimal places than allowed, attempt to increase FMT's + width until that number of decimal places can be achieved. */ + if (fmt->d > fmt_max_decimals (fmt->type, fmt->w, use) + && fmt_takes_decimals (fmt->type)) { - int w; - for (w = fmt->w; w <= max_w; w++) - if (fmt_max_decimals (fmt->type, w, for_input) >= fmt->d) - { - fmt->w = w; - break; - } + int max_w = fmt_max_width (fmt->type, use); + for (; fmt->w < max_w; fmt->w++) + if (fmt->d <= fmt_max_decimals (fmt->type, fmt->w, use)) + break; } /* Clamp decimals to those allowed by format and width. */ - max_d = fmt_max_decimals (fmt->type, fmt->w, for_input); - if (fmt->d < 0) - fmt->d = 0; - else if (fmt->d > max_d) - fmt->d = max_d; + fmt_clamp_decimals (fmt, use); } /* Adjusts FMT's width and decimal places to be valid for an @@ -538,7 +532,7 @@ fmt_fix (struct fmt_spec *fmt, bool for_input) void fmt_fix_input (struct fmt_spec *fmt) { - fmt_fix (fmt, true); + fmt_fix (fmt, FMT_FOR_INPUT); } /* Adjusts FMT's width and decimal places to be valid for an @@ -546,7 +540,27 @@ fmt_fix_input (struct fmt_spec *fmt) void fmt_fix_output (struct fmt_spec *fmt) { - fmt_fix (fmt, false); + fmt_fix (fmt, FMT_FOR_OUTPUT); +} + +/* Sets FMT's width to WIDTH (or the nearest width allowed by FMT's type) and + reduces its decimal places as necessary (if necessary) for that width. */ +void +fmt_change_width (struct fmt_spec *fmt, int width, enum fmt_use use) +{ + fmt->w = width; + fmt_clamp_width (fmt, use); + fmt_clamp_decimals (fmt, use); +} + +/* Sets FMT's decimal places to DECIMALS (or the nearest number of decimal + places allowed by FMT's type) and increases its width as necessary (if + necessary) for that number of decimal places. */ +void +fmt_change_decimals (struct fmt_spec *fmt, int decimals, enum fmt_use use) +{ + fmt->d = decimals; + fmt_fix (fmt, use); } /* Describes a display format. */ @@ -576,7 +590,7 @@ fmt_from_name (const char *name, enum fmt_type *type) int i; for (i = 0; i < FMT_NUMBER_OF_FORMATS; i++) - if (!strcasecmp (name, get_fmt_desc (i)->name)) + if (!c_strcasecmp (name, get_fmt_desc (i)->name)) { *type = i; return true; @@ -592,20 +606,20 @@ fmt_takes_decimals (enum fmt_type type) return fmt_max_output_decimals (type, fmt_max_output_width (type)) > 0; } -/* Returns the minimum width of the given format TYPE, - for input if FOR_INPUT is true, - for output otherwise. */ +/* Returns the minimum width of the given format TYPE for the given USE. */ int -fmt_min_width (enum fmt_type type, bool for_input) +fmt_min_width (enum fmt_type type, enum fmt_use use) { - return for_input ? fmt_min_input_width (type) : fmt_min_output_width (type); + return (use == FMT_FOR_INPUT + ? fmt_min_input_width (type) + : fmt_min_output_width (type)); } /* Returns the maximum width of the given format TYPE, for input if FOR_INPUT is true, for output otherwise. */ int -fmt_max_width (enum fmt_type type, bool for_input UNUSED) +fmt_max_width (enum fmt_type type, enum fmt_use use UNUSED) { /* Maximum width is actually invariant of whether the format is for input or output, so FOR_INPUT is unused. */ @@ -635,11 +649,9 @@ fmt_max_width (enum fmt_type type, bool for_input UNUSED) } /* Returns the maximum number of decimal places allowed for the - given format TYPE with a width of WIDTH places, - for input if FOR_INPUT is true, - for output otherwise. */ + given format TYPE with a width of WIDTH places, for the given USE. */ int -fmt_max_decimals (enum fmt_type type, int width, bool for_input) +fmt_max_decimals (enum fmt_type type, int width, enum fmt_use use) { int max_d; @@ -648,16 +660,16 @@ fmt_max_decimals (enum fmt_type type, int width, bool for_input) case FMT_F: case FMT_COMMA: case FMT_DOT: - max_d = for_input ? width : width - 1; + max_d = use == FMT_FOR_INPUT ? width : width - 1; break; case FMT_DOLLAR: case FMT_PCT: - max_d = for_input ? width : width - 2; + max_d = use == FMT_FOR_INPUT ? width : width - 2; break; case FMT_E: - max_d = for_input ? width : width - 7; + max_d = use == FMT_FOR_INPUT ? width : width - 7; break; case FMT_CCA: @@ -665,7 +677,7 @@ fmt_max_decimals (enum fmt_type type, int width, bool for_input) case FMT_CCC: case FMT_CCD: case FMT_CCE: - assert (!for_input); + assert (use == FMT_FOR_OUTPUT); max_d = width - 1; break; @@ -711,6 +723,14 @@ fmt_max_decimals (enum fmt_type type, int width, bool for_input) max_d = width - 21; break; + case FMT_YMDHMS: + max_d = width - 20; + break; + + case FMT_MTIME: + max_d = width - 6; + break; + case FMT_TIME: max_d = width - 9; break; @@ -750,7 +770,7 @@ fmt_min_input_width (enum fmt_type type) int fmt_max_input_width (enum fmt_type type) { - return fmt_max_width (type, true); + return fmt_max_width (type, FMT_FOR_INPUT); } /* Returns the maximum number of decimal places allowed in an @@ -759,7 +779,7 @@ int fmt_max_input_decimals (enum fmt_type type, int width) { assert (valid_width (type, width, true)); - return fmt_max_decimals (type, width, true); + return fmt_max_decimals (type, width, FMT_FOR_INPUT); } /* Returns the minimum acceptable width for an output field @@ -775,7 +795,7 @@ fmt_min_output_width (enum fmt_type type) int fmt_max_output_width (enum fmt_type type) { - return fmt_max_width (type, false); + return fmt_max_width (type, FMT_FOR_OUTPUT); } /* Returns the maximum number of decimal places allowed in an @@ -784,7 +804,7 @@ int fmt_max_output_decimals (enum fmt_type type, int width) { assert (valid_width (type, width, false)); - return fmt_max_decimals (type, width, false); + return fmt_max_decimals (type, width, FMT_FOR_OUTPUT); } /* Returns the width step for a field formatted with the given @@ -869,6 +889,40 @@ fmt_from_io (int io, enum fmt_type *fmt_type) } } +/* Translate U32, which is in the form found in SAV and SPV files, into a + format specification, and stores the new specification in *F. + + If LOOSE is false, checks that the format specification is appropriate as an + output format for a variable with the given WIDTH and reports an error if + not. If LOOSE is true, instead adjusts the format's width and decimals as + necessary to be suitable. + + Return true if successful, false if there was an error.. */ +bool +fmt_from_u32 (uint32_t u32, int width, bool loose, struct fmt_spec *f) +{ + uint8_t raw_type = u32 >> 16; + uint8_t w = u32 >> 8; + uint8_t d = u32; + + msg_disable (); + f->w = w; + f->d = d; + bool ok = fmt_from_io (raw_type, &f->type); + if (ok) + { + if (loose) + fmt_fix_output (f); + else + ok = fmt_check_output (f); + } + if (ok) + ok = fmt_check_width_compat (f, width); + msg_enable (); + + return ok; +} + /* Returns true if TYPE may be used as an input format, false otherwise. */ bool @@ -878,38 +932,91 @@ fmt_usable_for_input (enum fmt_type type) return fmt_get_category (type) != FMT_CAT_CUSTOM; } -/* For time and date formats, returns a template used for input - and output. */ +/* For time and date formats, returns a template used for input and output in a + field of the given WIDTH. + + WIDTH only affects whether a 2-digit year or a 4-digit year is used, that + is, whether the returned string contains "yy" or "yyyy", and whether seconds + are include, that is, whether the returned string contains ":SS". A caller + that doesn't care whether the returned string contains "yy" or "yyyy" or + ":SS" can just specify 0 to omit them. */ const char * -fmt_date_template (enum fmt_type type) +fmt_date_template (enum fmt_type type, int width) { + const char *s1, *s2; + switch (type) { case FMT_DATE: - return "dd-mmm-yy"; + s1 = "dd-mmm-yy"; + s2 = "dd-mmm-yyyy"; + break; + case FMT_ADATE: - return "mm/dd/yy"; + s1 = "mm/dd/yy"; + s2 = "mm/dd/yyyy"; + break; + case FMT_EDATE: - return "dd.mm.yy"; + s1 = "dd.mm.yy"; + s2 = "dd.mm.yyyy"; + break; + case FMT_JDATE: - return "yyddd"; + s1 = "yyddd"; + s2 = "yyyyddd"; + break; + case FMT_SDATE: - return "yy/mm/dd"; + s1 = "yy/mm/dd"; + s2 = "yyyy/mm/dd"; + break; + case FMT_QYR: - return "q Q yy"; + s1 = "q Q yy"; + s2 = "q Q yyyy"; + break; + case FMT_MOYR: - return "mmmXyy"; + s1 = "mmm yy"; + s2 = "mmm yyyy"; + break; + case FMT_WKYR: - return "ww WK yy"; + s1 = "ww WK yy"; + s2 = "ww WK yyyy"; + break; + case FMT_DATETIME: - return "dd-mmm-yyyy HH:MM"; + s1 = "dd-mmm-yyyy HH:MM"; + s2 = "dd-mmm-yyyy HH:MM:SS"; + break; + + case FMT_YMDHMS: + s1 = "yyyy-mm-dd HH:MM"; + s2 = "yyyy-mm-dd HH:MM:SS"; + break; + + case FMT_MTIME: + s1 = "MM"; + s2 = "MM:SS"; + break; + case FMT_TIME: - return "H:MM"; + s1 = "HH:MM"; + s2 = "HH:MM:SS"; + break; + case FMT_DTIME: - return "D HH:MM"; + s1 = "D HH:MM"; + s2 = "D HH:MM:SS"; + break; + default: NOT_REACHED (); } + + return width >= strlen (s2) ? s2 : s1; } /* Returns a string representing the format TYPE for use in a GUI dialog. */ @@ -939,6 +1046,8 @@ fmt_gui_name (enum fmt_type type) case FMT_MOYR: case FMT_WKYR: case FMT_DATETIME: + case FMT_YMDHMS: + case FMT_MTIME: case FMT_TIME: case FMT_DTIME: case FMT_WKDAY: @@ -972,14 +1081,12 @@ is_fmt_type (enum fmt_type type) } /* Returns true if WIDTH is a valid width for the given format - TYPE, - for input if FOR_INPUT is true, - for output otherwise. */ + TYPE, for the given USE. */ static bool -valid_width (enum fmt_type type, int width, bool for_input) +valid_width (enum fmt_type type, int width, enum fmt_use use) { - return (width >= fmt_min_width (type, for_input) - && width <= fmt_max_width (type, for_input)); + return (width >= fmt_min_width (type, use) + && width <= fmt_max_width (type, use)); } /* Returns the maximum number of decimal digits in an unsigned @@ -991,13 +1098,46 @@ max_digits_for_bytes (int bytes) assert (bytes > 0 && bytes <= sizeof map / sizeof *map); return map[bytes - 1]; } - -/* Sets AFFIX's string value to S, a UTF-8 encoded string. */ + +/* Clamp FMT's width to the range and values allowed by FMT's type. */ static void -fmt_affix_set (struct fmt_affix *affix, const char *s) +fmt_clamp_width (struct fmt_spec *fmt, enum fmt_use use) { - affix->s = s[0] == '\0' ? CONST_CAST (char *, "") : xstrdup (s); - affix->width = u8_strwidth (CHAR_CAST (const uint8_t *, s), "UTF-8"); + unsigned int step; + int min_w, max_w; + + min_w = fmt_min_width (fmt->type, use); + max_w = fmt_max_width (fmt->type, use); + if (fmt->w < min_w) + fmt->w = min_w; + else if (fmt->w > max_w) + fmt->w = max_w; + + /* Round width to step. */ + step = fmt_step_width (fmt->type); + fmt->w = ROUND_DOWN (fmt->w, step); +} + +/* Clamp FMT's decimal places to the range allowed by FMT's type and width. */ +static void +fmt_clamp_decimals (struct fmt_spec *fmt, enum fmt_use use) +{ + int max_d; + + max_d = fmt_max_decimals (fmt->type, fmt->w, use); + if (fmt->d < 0) + fmt->d = 0; + else if (fmt->d > max_d) + fmt->d = max_d; +} + +static struct fmt_affix +fmt_affix_clone (const struct fmt_affix *old) +{ + return (struct fmt_affix) { + .s = old->s ? xstrdup (old->s) : NULL, + .width = old->width, + }; } /* Frees data in AFFIX. */ @@ -1008,32 +1148,99 @@ fmt_affix_free (struct fmt_affix *affix) free (affix->s); } -static void -fmt_number_style_init (struct fmt_number_style *style) +/* Find and returns the grouping character in CC_STRING (either '.' or ',') or + 0 on error. */ +static int +find_cc_separators (const char *cc_string) { - fmt_affix_set (&style->neg_prefix, ""); - fmt_affix_set (&style->prefix, ""); - fmt_affix_set (&style->suffix, ""); - fmt_affix_set (&style->neg_suffix, ""); - style->decimal = '.'; - style->grouping = 0; + /* Count commas and periods. There must be exactly three of + one or the other, except that an apostrophe escapes a + following comma or period. */ + int n_commas = 0; + int n_dots = 0; + for (const char *p = cc_string; *p; p++) + if (*p == ',') + n_commas++; + else if (*p == '.') + n_dots++; + else if (*p == '\'' && (p[1] == '.' || p[1] == ',' || p[1] == '\'')) + p++; + + return (n_commas == 3 ? (n_dots != 3 ? ',' : 0) + : n_dots == 3 ? '.' + : 0); } -static void -fmt_number_style_clone (struct fmt_number_style *new, - const struct fmt_number_style *old) +/* Extracts a token from IN into a newly allocated string AFFIXP. Tokens are + delimited by GROUPING. Returns the first character following the token. */ +static struct fmt_affix +extract_cc_token (const char **sp, int grouping, size_t *extra_bytes) { - fmt_affix_set (&new->neg_prefix, old->neg_prefix.s); - fmt_affix_set (&new->prefix, old->prefix.s); - fmt_affix_set (&new->suffix, old->suffix.s); - fmt_affix_set (&new->neg_suffix, old->neg_suffix.s); - new->decimal = old->decimal; - new->grouping = old->grouping; - new->extra_bytes = old->extra_bytes; + const char *p = *sp; + for (; *p && *p != grouping; p++) + if (*p == '\'' && p[1] == grouping) + p++; + + size_t length = p - *sp; + char *affix = xmemdup0 (*sp, length); + size_t width = u8_strwidth (CHAR_CAST (const uint8_t *, affix), "UTF-8"); + if (length > width) + *extra_bytes += length - width; + + *sp = p + (*p != 0); + + return (struct fmt_affix) { .s = affix, .width = width }; +} + +struct fmt_number_style * +fmt_number_style_from_string (const char *s) +{ + char grouping = find_cc_separators (s); + if (!grouping) + return NULL; + + size_t extra_bytes = 0; + struct fmt_affix neg_prefix = extract_cc_token (&s, grouping, &extra_bytes); + struct fmt_affix prefix = extract_cc_token (&s, grouping, &extra_bytes); + struct fmt_affix suffix = extract_cc_token (&s, grouping, &extra_bytes); + struct fmt_affix neg_suffix = extract_cc_token (&s, grouping, &extra_bytes); + + struct fmt_number_style *style = xmalloc (sizeof *style); + *style = (struct fmt_number_style) { + .neg_prefix = neg_prefix, + .prefix = prefix, + .suffix = suffix, + .neg_suffix = neg_suffix, + .decimal = grouping == '.' ? ',' : '.', + .grouping = grouping, + .extra_bytes = extra_bytes, + }; + return style; +} + +struct fmt_number_style * +fmt_number_style_clone (const struct fmt_number_style *old) +{ + if (old) + { + struct fmt_number_style *new = xmalloc (sizeof *new); + *new = (struct fmt_number_style) { + .neg_prefix = fmt_affix_clone (&old->neg_prefix), + .prefix = fmt_affix_clone (&old->prefix), + .suffix = fmt_affix_clone (&old->suffix), + .neg_suffix = fmt_affix_clone (&old->neg_suffix), + .decimal = old->decimal, + .grouping = old->grouping, + .extra_bytes = old->extra_bytes, + }; + return new; + } + else + return NULL; } /* Destroys a struct fmt_number_style. */ -static void +void fmt_number_style_destroy (struct fmt_number_style *style) { if (style != NULL) @@ -1042,6 +1249,7 @@ fmt_number_style_destroy (struct fmt_number_style *style) fmt_affix_free (&style->prefix); fmt_affix_free (&style->suffix); fmt_affix_free (&style->neg_suffix); + free (style); } } @@ -1077,3 +1285,6 @@ get_fmt_desc (enum fmt_type type) } const struct fmt_spec F_8_0 = {FMT_F, 8, 0}; +const struct fmt_spec F_8_2 = {FMT_F, 8, 2}; +const struct fmt_spec F_4_3 = {FMT_F, 4, 3}; +const struct fmt_spec F_5_1 = {FMT_F, 5, 1};