From d8e1480a27e1e26b8dfa97681bf8e7f90946d336 Mon Sep 17 00:00:00 2001 From: Hisham Muhammad Date: Mon, 22 Nov 2010 12:40:20 +0000 Subject: Remove arbitrary limit from rich strings Fix subtree hiding Fix reading of CPU values in hidden threads Fix hiding of zombie processes as kernel threads Remove "debug proc" code Code cleanup in processElements --- RichString.c | 125 +++++++++++++++++++++++++++++++++++------------------------ 1 file changed, 74 insertions(+), 51 deletions(-) (limited to 'RichString.c') diff --git a/RichString.c b/RichString.c index a62a053a..6d87cd84 100644 --- a/RichString.c +++ b/RichString.c @@ -22,26 +22,31 @@ /*{ -#define RichString_init(this) (this)->len = 0 -#define RichString_initVal(this) (this).len = 0 +#define RichString_size(this) ((this)->chlen) +#define RichString_sizeVal(this) ((this).chlen) + +#define RichString_begin(this) RichString (this); (this).chlen = 0; (this).chptr = (this).chstr; +#define RichString_beginAllocated(this) (this).chlen = 0; (this).chptr = (this).chstr; +#define RichString_end(this) RichString_prune(&(this)); #ifdef HAVE_LIBNCURSESW -#define RichString_printVal(this, y, x) mvadd_wchstr(y, x, this.chstr) -#define RichString_printoffnVal(this, y, x, off, n) mvadd_wchnstr(y, x, this.chstr + off, n) -#define RichString_getCharVal(this, i) (this.chstr[i].chars[0] & 255) +#define RichString_printVal(this, y, x) mvadd_wchstr(y, x, (this).chptr) +#define RichString_printoffnVal(this, y, x, off, n) mvadd_wchnstr(y, x, (this).chptr + off, n) +#define RichString_getCharVal(this, i) ((this).chptr[i].chars[0] & 255) +#define RichString_setChar(this, at, ch) do{ (this)->chptr[(at)].chars[0] = ch; } while(0) +#define CharType cchar_t #else -#define RichString_printVal(this, y, x) mvaddchstr(y, x, this.chstr) -#define RichString_printoffnVal(this, y, x, off, n) mvaddchnstr(y, x, this.chstr + off, n) -#define RichString_getCharVal(this, i) (this.chstr[i]) +#define RichString_printVal(this, y, x) mvaddchstr(y, x, (this).chptr) +#define RichString_printoffnVal(this, y, x, off, n) mvaddchnstr(y, x, (this).chptr + off, n) +#define RichString_getCharVal(this, i) ((this).chptr[i]) +#define RichString_setChar(this, at, ch) do{ (this)->chptr[(at)] = ch; } while(0) +#define CharType chtype #endif typedef struct RichString_ { - int len; -#ifdef HAVE_LIBNCURSESW - cchar_t chstr[RICHSTRING_MAXLEN+1]; -#else - chtype chstr[RICHSTRING_MAXLEN+1]; -#endif + int chlen; + CharType chstr[RICHSTRING_MAXLEN+1]; + CharType* chptr; } RichString; }*/ @@ -50,35 +55,57 @@ typedef struct RichString_ { #define MIN(a,b) ((a)<(b)?(a):(b)) #endif +#define charBytes(n) (sizeof(CharType) * (n)) + +static inline void RichString_setLen(RichString* this, int len) { + if (this->chlen <= RICHSTRING_MAXLEN) { + if (len > RICHSTRING_MAXLEN) { + this->chptr = malloc(charBytes(len+1)); + memcpy(this->chptr, this->chstr, charBytes(this->chlen+1)); + } + } else { + if (len <= RICHSTRING_MAXLEN) { + memcpy(this->chstr, this->chptr, charBytes(this->chlen)); + free(this->chptr); + this->chptr = this->chstr; + } else { + this->chptr = realloc(this->chptr, charBytes(len+1)); + } + } + RichString_setChar(this, len, 0); + this->chlen = len; +} + #ifdef HAVE_LIBNCURSESW inline void RichString_appendn(RichString* this, int attrs, const char* data_c, int len) { - wchar_t data[RICHSTRING_MAXLEN]; - len = mbstowcs(data, data_c, RICHSTRING_MAXLEN); + wchar_t data[len+1]; + len = mbstowcs(data, data_c, len); if (len<0) return; - int last = MIN(RICHSTRING_MAXLEN - 1, len + this->len); - for (int i = this->len, j = 0; i < last; i++, j++) { - memset(&this->chstr[i], 0, sizeof(this->chstr[i])); - this->chstr[i].chars[0] = data[j]; - this->chstr[i].attr = attrs; + int oldLen = this->chlen; + int newLen = len + oldLen; + RichString_setLen(this, newLen); + for (int i = oldLen, j = 0; i < newLen; i++, j++) { + memset(&this->chptr[i], 0, sizeof(this->chptr[i])); + this->chptr[i].chars[0] = data[j]; + this->chptr[i].attr = attrs; } - this->chstr[last].chars[0] = 0; - this->len = last; + this->chptr[newLen].chars[0] = 0; } -inline void RichString_setAttrn(RichString *this, int attrs, int start, int finish) { - cchar_t* ch = this->chstr + start; +inline void RichString_setAttrn(RichString* this, int attrs, int start, int finish) { + cchar_t* ch = this->chptr + start; for (int i = start; i <= finish; i++) { ch->attr = attrs; ch++; } } -int RichString_findChar(RichString *this, char c, int start) { +int RichString_findChar(RichString* this, char c, int start) { wchar_t wc = btowc(c); - cchar_t* ch = this->chstr + start; - for (int i = start; i < this->len; i++) { + cchar_t* ch = this->chptr + start; + for (int i = start; i < this->chlen; i++) { if (ch->chars[0] == wc) return i; ch++; @@ -89,25 +116,25 @@ int RichString_findChar(RichString *this, char c, int start) { #else inline void RichString_appendn(RichString* this, int attrs, const char* data_c, int len) { - int last = MIN(RICHSTRING_MAXLEN - 1, len + this->len); - for (int i = this->len, j = 0; i < last; i++, j++) - this->chstr[i] = (isprint(data_c[j]) ? data_c[j] : '?') | attrs; - - this->chstr[last] = 0; - this->len = last; + int oldLen = this->chlen; + int newLen = len + oldLen; + RichString_setLen(this, newLen); + for (int i = oldLen, j = 0; i < newLen; i++, j++) + this->chptr[i] = (isprint(data_c[j]) ? data_c[j] : '?') | attrs; + this->chptr[newLen] = 0; } -void RichString_setAttrn(RichString *this, int attrs, int start, int finish) { - chtype* ch = this->chstr + start; +void RichString_setAttrn(RichString* this, int attrs, int start, int finish) { + chtype* ch = this->chptr + start; for (int i = start; i <= finish; i++) { *ch = (*ch & 0xff) | attrs; ch++; } } -int RichString_findChar(RichString *this, char c, int start) { - chtype* ch = this->chstr + start; - for (int i = start; i < this->len; i++) { +int RichString_findChar(RichString* this, char c, int start) { + chtype* ch = this->chptr + start; + for (int i = start; i < this->chlen; i++) { if ((*ch & 0xff) == (chtype) c) return i; ch++; @@ -118,11 +145,14 @@ int RichString_findChar(RichString *this, char c, int start) { #endif void RichString_prune(RichString* this) { - this->len = 0; + if (this->chlen > RICHSTRING_MAXLEN) + free(this->chptr); + this->chptr = this->chstr; + this->chlen = 0; } -void RichString_setAttr(RichString *this, int attrs) { - RichString_setAttrn(this, attrs, 0, this->len - 1); +void RichString_setAttr(RichString* this, int attrs) { + RichString_setAttrn(this, attrs, 0, this->chlen - 1); } inline void RichString_append(RichString* this, int attrs, const char* data) { @@ -130,13 +160,6 @@ inline void RichString_append(RichString* this, int attrs, const char* data) { } void RichString_write(RichString* this, int attrs, const char* data) { - RichString_init(this); - RichString_append(this, attrs, data); -} - -RichString RichString_quickString(int attrs, const char* data) { - RichString str; - RichString_initVal(str); - RichString_write(&str, attrs, data); - return str; + RichString_setLen(this, 0); + RichString_appendn(this, attrs, data, strlen(data)); } -- cgit v1.2.3