Page MenuHomeFreeBSD
Files F145224226

D30170.id88989.diff
No OneTemporary
Actions

D30170.id88989.diff
View Options

diff --git a/usr.bin/sort/bwstring.h b/usr.bin/sort/bwstring.h
--- a/usr.bin/sort/bwstring.h
+++ b/usr.bin/sort/bwstring.h
@@ -38,6 +38,7 @@
#include <sysexits.h>
#include <wchar.h>
+#include "sort.h"
#include "mem.h"
extern bool byte_sort;
@@ -109,7 +110,7 @@
bws_end(struct bwstring *bws)
{
- return ((MB_CUR_MAX == 1) ?
+ return ((mb_cur_max == 1) ?
(bwstring_iterator) (bws->data.cstr + bws->len) :
(bwstring_iterator) (bws->data.wstr + bws->len));
}
@@ -118,7 +119,7 @@
bws_iterator_inc(bwstring_iterator iter, size_t pos)
{
- if (MB_CUR_MAX == 1)
+ if (mb_cur_max == 1)
return ((unsigned char *) iter) + pos;
else
return ((wchar_t*) iter) + pos;
@@ -128,7 +129,7 @@
bws_get_iter_value(bwstring_iterator iter)
{
- if (MB_CUR_MAX == 1)
+ if (mb_cur_max == 1)
return *((unsigned char *) iter);
else
return *((wchar_t*) iter);
@@ -137,7 +138,7 @@
int
bws_iterator_cmp(bwstring_iterator iter1, bwstring_iterator iter2, size_t len);
-#define BWS_GET(bws, pos) ((MB_CUR_MAX == 1) ? ((bws)->data.cstr[(pos)]) : (bws)->data.wstr[(pos)])
+#define BWS_GET(bws, pos) ((mb_cur_max == 1) ? ((bws)->data.cstr[(pos)]) : (bws)->data.wstr[(pos)])
void initialise_months(void);
diff --git a/usr.bin/sort/bwstring.c b/usr.bin/sort/bwstring.c
--- a/usr.bin/sort/bwstring.c
+++ b/usr.bin/sort/bwstring.c
@@ -59,7 +59,7 @@
char *tmp;
size_t len;
- if (MB_CUR_MAX == 1) {
+ if (mb_cur_max == 1) {
if (cmonths == NULL) {
char *m;
@@ -143,7 +143,7 @@
bwsprintf(FILE *f, struct bwstring *bws, const char *prefix, const char *suffix)
{
- if (MB_CUR_MAX == 1)
+ if (mb_cur_max == 1)
fprintf(f, "%s%s%s", prefix, bws->data.cstr, suffix);
else
fprintf(f, "%s%S%s", prefix, bws->data.wstr, suffix);
@@ -158,14 +158,14 @@
size_t bwsrawlen(const struct bwstring *bws)
{
- return ((MB_CUR_MAX == 1) ? bws->len : SIZEOF_WCHAR_STRING(bws->len));
+ return ((mb_cur_max == 1) ? bws->len : SIZEOF_WCHAR_STRING(bws->len));
}
size_t
bws_memsize(const struct bwstring *bws)
{
- return ((MB_CUR_MAX == 1) ? (bws->len + 2 + sizeof(struct bwstring)) :
+ return ((mb_cur_max == 1) ? (bws->len + 2 + sizeof(struct bwstring)) :
(SIZEOF_WCHAR_STRING(bws->len + 1) + sizeof(struct bwstring)));
}
@@ -175,7 +175,7 @@
if (bws && newlen != bws->len && newlen <= bws->len) {
bws->len = newlen;
- if (MB_CUR_MAX == 1)
+ if (mb_cur_max == 1)
bws->data.cstr[newlen] = '\0';
else
bws->data.wstr[newlen] = L'\0';
@@ -190,14 +190,14 @@
{
struct bwstring *ret;
- if (MB_CUR_MAX == 1)
+ if (mb_cur_max == 1)
ret = sort_malloc(sizeof(struct bwstring) + 1 + sz);
else
ret = sort_malloc(sizeof(struct bwstring) +
SIZEOF_WCHAR_STRING(sz + 1));
ret->len = sz;
- if (MB_CUR_MAX == 1)
+ if (mb_cur_max == 1)
ret->data.cstr[ret->len] = '\0';
else
ret->data.wstr[ret->len] = L'\0';
@@ -218,7 +218,7 @@
else {
struct bwstring *ret = bwsalloc(s->len);
- if (MB_CUR_MAX == 1)
+ if (mb_cur_max == 1)
memcpy(ret->data.cstr, s->data.cstr, (s->len));
else
memcpy(ret->data.wstr, s->data.wstr,
@@ -242,7 +242,7 @@
ret = bwsalloc(len);
- if (MB_CUR_MAX == 1)
+ if (mb_cur_max == 1)
for (size_t i = 0; i < len; ++i)
ret->data.cstr[i] = (unsigned char) str[i];
else
@@ -263,7 +263,7 @@
ret = bwsalloc(len);
if (str) {
- if (MB_CUR_MAX == 1)
+ if (mb_cur_max == 1)
memcpy(ret->data.cstr, str, len);
else {
mbstate_t mbs;
@@ -277,7 +277,7 @@
memset(&mbs, 0, sizeof(mbs));
while (cptr < len) {
- size_t n = MB_CUR_MAX;
+ size_t n = mb_cur_max;
if (n > len - cptr)
n = len - cptr;
@@ -334,7 +334,7 @@
nums = dst->len;
dst->len = nums;
- if (MB_CUR_MAX == 1) {
+ if (mb_cur_max == 1) {
memcpy(dst->data.cstr, src->data.cstr, nums);
dst->data.cstr[dst->len] = '\0';
} else {
@@ -363,7 +363,7 @@
nums = size;
dst->len = nums;
- if (MB_CUR_MAX == 1) {
+ if (mb_cur_max == 1) {
memcpy(dst->data.cstr, src->data.cstr, nums);
dst->data.cstr[dst->len] = '\0';
} else {
@@ -398,7 +398,7 @@
if (nums > size)
nums = size;
dst->len = nums;
- if (MB_CUR_MAX == 1) {
+ if (mb_cur_max == 1) {
memcpy(dst->data.cstr, src->data.cstr + offset,
(nums));
dst->data.cstr[dst->len] = '\0';
@@ -420,7 +420,7 @@
bwsfwrite(struct bwstring *bws, FILE *f, bool zero_ended)
{
- if (MB_CUR_MAX == 1) {
+ if (mb_cur_max == 1) {
size_t len = bws->len;
if (!zero_ended) {
@@ -478,7 +478,7 @@
eols = zero_ended ? btowc('\0') : btowc('\n');
- if (!zero_ended && (MB_CUR_MAX > 1)) {
+ if (!zero_ended && (mb_cur_max > 1)) {
wchar_t *ret;
ret = fgetwln(f, len);
@@ -494,7 +494,7 @@
}
return (bwssbdup(ret, *len));
- } else if (!zero_ended && (MB_CUR_MAX == 1)) {
+ } else if (!zero_ended && (mb_cur_max == 1)) {
char *ret;
ret = fgetln(f, len);
@@ -523,7 +523,7 @@
}
rb->fgetwln_z_buffer[*len] = 0;
- if (MB_CUR_MAX == 1)
+ if (mb_cur_max == 1)
while (!feof(f)) {
int c;
@@ -603,7 +603,7 @@
if (len < cmp_len)
cmp_len = len;
- if (MB_CUR_MAX == 1) {
+ if (mb_cur_max == 1) {
const unsigned char *s1, *s2;
s1 = bws1->data.cstr + offset;
@@ -696,7 +696,7 @@
len1 -= offset;
len2 -= offset;
- if (MB_CUR_MAX == 1) {
+ if (mb_cur_max == 1) {
const unsigned char *s1, *s2;
s1 = bws1->data.cstr + offset;
@@ -852,7 +852,7 @@
{
double ret = 0;
- if (MB_CUR_MAX == 1) {
+ if (mb_cur_max == 1) {
unsigned char *end, *s;
char *ep;
@@ -909,7 +909,7 @@
bws_month_score(const struct bwstring *s0)
{
- if (MB_CUR_MAX == 1) {
+ if (mb_cur_max == 1) {
const unsigned char *end, *s;
s = s0->data.cstr;
@@ -949,7 +949,7 @@
ignore_leading_blanks(struct bwstring *str)
{
- if (MB_CUR_MAX == 1) {
+ if (mb_cur_max == 1) {
unsigned char *dst, *end, *src;
src = str->data.cstr;
@@ -1005,7 +1005,7 @@
{
size_t newlen = str->len;
- if (MB_CUR_MAX == 1) {
+ if (mb_cur_max == 1) {
unsigned char *dst, *end, *src;
unsigned char c;
@@ -1058,7 +1058,7 @@
{
size_t newlen = str->len;
- if (MB_CUR_MAX == 1) {
+ if (mb_cur_max == 1) {
unsigned char *dst, *end, *src;
unsigned char c;
@@ -1109,7 +1109,7 @@
ignore_case(struct bwstring *str)
{
- if (MB_CUR_MAX == 1) {
+ if (mb_cur_max == 1) {
unsigned char *end, *s;
s = str->data.cstr;
@@ -1137,7 +1137,7 @@
bws_disorder_warnx(struct bwstring *s, const char *fn, size_t pos)
{
- if (MB_CUR_MAX == 1)
+ if (mb_cur_max == 1)
warnx("%s:%zu: disorder: %s", fn, pos + 1, s->data.cstr);
else
warnx("%s:%zu: disorder: %ls", fn, pos + 1, s->data.wstr);
diff --git a/usr.bin/sort/radixsort.c b/usr.bin/sort/radixsort.c
--- a/usr.bin/sort/radixsort.c
+++ b/usr.bin/sort/radixsort.c
@@ -258,7 +258,7 @@
static inline int
get_wc_index(struct sort_list_item *sli, size_t level)
{
- const size_t wcfact = (MB_CUR_MAX == 1) ? 1 : sizeof(wchar_t);
+ const size_t wcfact = (mb_cur_max == 1) ? 1 : sizeof(wchar_t);
const struct key_value *kv;
const struct bwstring *bws;
@@ -331,7 +331,7 @@
static void
run_sort_level_next(struct sort_level *sl)
{
- const size_t wcfact = (MB_CUR_MAX == 1) ? 1 : sizeof(wchar_t);
+ const size_t wcfact = (mb_cur_max == 1) ? 1 : sizeof(wchar_t);
struct sort_level *slc;
size_t i, sln, tosort_num;
diff --git a/usr.bin/sort/sort.h b/usr.bin/sort/sort.h
--- a/usr.bin/sort/sort.h
+++ b/usr.bin/sort/sort.h
@@ -127,4 +127,10 @@
extern struct sort_mods * const default_sort_mods;
+/*
+ * Cached value of MB_CUR_MAX. Because MB_CUR_MAX is used often throughout the program,
+ * this avoids frequent calls to __mb_cur_max.
+ */
+extern int mb_cur_max;
+
#endif /* __BSD_SORT_H__ */
diff --git a/usr.bin/sort/sort.c b/usr.bin/sort/sort.c
--- a/usr.bin/sort/sort.c
+++ b/usr.bin/sort/sort.c
@@ -99,6 +99,8 @@
bool debug_sort;
bool need_hint;
+int mb_cur_max;
+
#if defined(SORT_THREADS)
unsigned int ncpu = 1;
size_t nthreads = 1;
@@ -305,7 +307,7 @@
if (wc && c) {
int res;
- res = mbtowc(wc, c, MB_CUR_MAX);
+ res = mbtowc(wc, c, mb_cur_max);
if (res < 1)
*wc = def;
}
@@ -322,6 +324,8 @@
setlocale(LC_ALL, "");
+ mb_cur_max = MB_CUR_MAX;
+
lc = localeconv();
if (lc) {

File Metadata

Mime Type
text/plain
Expires
Wed, Feb 18, 7:08 AM (3 h, 35 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
28841006
Default Alt Text
D30170.id88989.diff (8 KB)

Event Timeline