Instead of beating the code with typecasts, express the operations the

"right" way.

src/sbbs3/js_global.c

@@ -1537,9 +1537,9 @@ js_html_encode(JSContext *cx, uintN argc, jsval *arglist)
 					j--;
 				break;
 			default:
-				if(((uchar)inbuf[i])&0x80) {
+				if(inbuf[i]&0x80) {
 					if(exascii) {
-						ch=((uchar)inbuf[i])^0x80;
+						ch=inbuf[i]^'\x80';
 						if(exasctbl[ch].name!=NULL)
 							j+=sprintf(tmpbuf+j,"&%s;",exasctbl[ch].name);
 						else