FEAT: OnTheSpot editing.
FEAT: tabbed windows (hey, just use screen...)
+4.5 Mon Dec 13 07:47:16 CET 2004
+ - fix an extremely nasty bug in utf8-conversion. upgrade is advised.
- completely re-wrote selection pasting. This should fix garbled
incomplete characters at 32k boundaries and too-short INCR
selections, and more. Increases memory requirements considerably
unsigned char *s = (unsigned char *)str;
- while (len--)
+ while (len)
{
if (s[0] < 0x80)
- *p++ = *s++;
+ {
+ *p++ = *s++; len--;
+ }
else if (len > 0
&& s[0] >= 0xc2 && s[0] <= 0xdf
&& (s[1] & 0xc0) == 0x80)
{
*p++ = ((s[0] & 0x1f) << 6)
| (s[1] & 0x3f);
- s += 2;
+ s += 2; len -= 2;
}
else if (len > 1
&& ( (s[0] == 0xe0 && s[1] >= 0xa0 && s[1] <= 0xbf)
*p++ = ((s[0] & 0x0f) << 12)
| ((s[1] & 0x3f) << 6)
| (s[2] & 0x3f);
- s += 3;
+ s += 3; len -= 3;
}
else if (len > 2
&& ( (s[0] == 0xf0 && s[1] >= 0x90 && s[1] <= 0xbf)
| ((s[1] & 0x3f) << 12)
| ((s[2] & 0x3f) << 6)
| (s[3] & 0x3f);
- s += 4;
+ s += 4; len -= 4;
}
else
{
*p++ = 0xfffd;
- s++;
+ s++; len--;
}
}
XTextProperty ct;
if (XGetWindowProperty (display->display, win, prop,
- 0, (long)(PROP_SIZE / 4),
+ 0, PROP_SIZE / 4,
delete_prop, AnyPropertyType,
&ct.encoding, &ct.format,
&ct.nitems, &bytes_after,
{
// fetch and append remaining data
XTextProperty ct2;
- unsigned long bytes_after2;
if (XGetWindowProperty (display->display, win, prop,
- ct.nitems / 4, (long) (bytes_after + 3) / 4,
+ ct.nitems / 4, (bytes_after + 3) / 4,
delete_prop, AnyPropertyType,
&ct2.encoding, &ct2.format,
- &ct2.nitems, &bytes_after2,
+ &ct2.nitems, &bytes_after,
&ct2.value) != Success)
goto bailout;
// realloc should be compatible to XFree, here, and elsewhere, too
- ct.value = (unsigned char *)realloc (ct.value, ct.nitems + bytes_after);
- memcpy (ct.value + ct.nitems, ct2.value, ct2.nitems);
+ ct.value = (unsigned char *)realloc (ct.value, ct.nitems + ct2.nitems + 1);
+ memcpy (ct.value + ct.nitems, ct2.value, ct2.nitems + 1);
ct.nitems += ct2.nitems;
XFree (ct2.value);
}
- else if (delete_prop)
- XDeleteProperty (display->display, win, prop);
if (ct.value == 0)
goto bailout;