X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=src%2Fdata%2Fvalue.c;h=144f39b5241d1fb7b8b36021e57c23cea65a22ae;hb=d3e294c031bb767336435d2f0048994103fcd47a;hp=2341f0293c92aa90df7242e92a807a806649421f;hpb=5c3291dc396b795696e94f47780308fd7ace6fc4;p=pspp-builds.git diff --git a/src/data/value.c b/src/data/value.c index 2341f029..144f39b5 100644 --- a/src/data/value.c +++ b/src/data/value.c @@ -1,5 +1,5 @@ /* PSPP - a program for statistical analysis. - Copyright (C) 1997-9, 2000, 2009 Free Software Foundation, Inc. + Copyright (C) 1997-9, 2000, 2009, 2010, 2011 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 @@ -15,16 +15,18 @@ along with this program. If not, see . */ #include -#include -#include -#include -#include -#include -#include +#include "data/value.h" -#include "minmax.h" -#include "xalloc.h" +#include "data/val-type.h" +#include "data/variable.h" +#include "libpspp/hash-functions.h" +#include "libpspp/pool.h" +#include "libpspp/str.h" +#include "gl/unistr.h" + +#include "gl/minmax.h" +#include "gl/xalloc.h" /* Copies the contents of string value SRC with width SRC_WIDTH to string value DST with width DST_WIDTH. If SRC_WIDTH is @@ -44,7 +46,7 @@ value_copy_rpad (union value *dst, int dst_width, const union value *src, int src_width, char pad) { - buf_copy_rpad (value_str_rw (dst, dst_width), dst_width, + u8_buf_copy_rpad (value_str_rw (dst, dst_width), dst_width, value_str (src, src_width), src_width, pad); } @@ -62,10 +64,10 @@ value_copy_rpad (union value *dst, int dst_width, DST was initialized. Passing, e.g., a smaller value in order to modify only a prefix of DST will not work in every case. */ void -value_copy_str_rpad (union value *dst, int dst_width, const char *src, +value_copy_str_rpad (union value *dst, int dst_width, const uint8_t *src, char pad) { - value_copy_buf_rpad (dst, dst_width, src, strlen (src), pad); + value_copy_buf_rpad (dst, dst_width, src, u8_strlen (src), pad); } /* Copies the SRC_LEN bytes at SRC to string value DST with width @@ -81,9 +83,9 @@ value_copy_str_rpad (union value *dst, int dst_width, const char *src, to modify only a prefix of DST will not work in every case. */ void value_copy_buf_rpad (union value *dst, int dst_width, - const char *src, size_t src_len, char pad) + const uint8_t *src, size_t src_len, char pad) { - buf_copy_rpad (value_str_rw (dst, dst_width), dst_width, src, src_len, pad); + u8_buf_copy_rpad (value_str_rw (dst, dst_width), dst_width, src, src_len, pad); } /* Sets V to the system-missing value for data of the given @@ -91,10 +93,13 @@ value_copy_buf_rpad (union value *dst, int dst_width, void value_set_missing (union value *v, int width) { - if (width == 0) - v->f = SYSMIS; - else - memset (value_str_rw (v, width), ' ', width); + if (width != -1) + { + if (width == 0) + v->f = SYSMIS; + else + memset (value_str_rw (v, width), ' ', width); + } } /* Compares A and B, which both have the given WIDTH, and returns @@ -102,8 +107,8 @@ value_set_missing (union value *v, int width) int value_compare_3way (const union value *a, const union value *b, int width) { - return (width == 0 - ? (a->f < b->f ? -1 : a->f > b->f) + return (width == -1 ? 0 + : width == 0 ? (a->f < b->f ? -1 : a->f > b->f) : memcmp (value_str (a, width), value_str (b, width), width)); } @@ -112,8 +117,8 @@ value_compare_3way (const union value *a, const union value *b, int width) bool value_equal (const union value *a, const union value *b, int width) { - return (width == 0 - ? a->f == b->f + return (width == -1 ? true + : width == 0 ? a->f == b->f : !memcmp (value_str (a, width), value_str (b, width), width)); } @@ -122,8 +127,8 @@ value_equal (const union value *a, const union value *b, int width) unsigned int value_hash (const union value *value, int width, unsigned int basis) { - return (width == 0 - ? hash_double (value->f, basis) + return (width == -1 ? basis + : width == 0 ? hash_double (value->f, basis) : hash_bytes (value_str (value, width), width, basis)); } @@ -142,7 +147,7 @@ value_is_resizable (const union value *value, int old_width, int new_width) return false; else { - const char *str = value_str (value, old_width); + const uint8_t *str = value_str (value, old_width); int i; for (i = new_width; i < old_width; i++) @@ -189,8 +194,8 @@ value_needs_resize (int old_width, int new_width) anyway in hopes of saving memory.) */ return (old_width != new_width && (new_width > old_width - || old_width >= MIN_LONG_STRING - || new_width >= MIN_LONG_STRING)); + || old_width > MAX_SHORT_STRING + || new_width > MAX_SHORT_STRING)); } /* Same as value_init, except that memory for VALUE (if @@ -207,6 +212,22 @@ value_init_pool (struct pool *pool, union value *value, int width) value->long_string = pool_alloc_unaligned (pool, width); } +/* Same as value_clone(), except that memory for VALUE (if necessary) is + allocated from POOL and will be freed automatically when POOL is destroyed. + + VALUE must not be freed manually by calling value_destroy(). If it needs to + be resized, it must be done using value_resize_pool() instead of + value_resize(). */ +void +value_clone_pool (struct pool *pool, + union value *value, const union value *src, int width) +{ + if (width > MAX_SHORT_STRING) + value->long_string = pool_clone_unaligned (pool, src->long_string, width); + else + *value = *src; +} + /* Same as value_resize, except that VALUE must have been allocated from POOL using value_init_pool. @@ -222,7 +243,7 @@ value_resize_pool (struct pool *pool, union value *value, { if (new_width > MAX_SHORT_STRING) { - char *new_long_string = pool_alloc_unaligned (pool, new_width); + uint8_t *new_long_string = pool_alloc_unaligned (pool, new_width); memcpy (new_long_string, value_str (value, old_width), old_width); value->long_string = new_long_string; }