From sbbs@VERT to CVS commit on Monday, January 07, 2019 23:38:00
src/sbbs3 js_bbs.cpp 1.174 1.175 js_console.cpp 1.121 1.122 main.cpp 1.740 1.741 sbbs.h 1.498 1.499
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/sbbs/src/sbbs3
Modified Files:
js_bbs.cpp js_console.cpp main.cpp sbbs.h
Log Message:
Fix various JSDOCS_BUILD with gcc warnings.
warning: ISO C++ forbids converting a string constant to 'char*'
and comparision between signed/unsigned int.
This does *not* address the reported build issue:
In function `_start':
(.text+0x20): undefined reference to `main'
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
From rswindell@VERT to CVS commit on Saturday, May 04, 2019 01:04:00
Modified Files:
js_bbs.cpp js_com.c js_console.cpp js_file.c js_global.c
js_msgbase.c js_queue.c js_socket.c js_system.c js_user.c
main.cpp sbbs.h
Log Message:
Define and use a wrapper for JS_GetInstancePrivate(): js_GetClassPrivate()
Use this in place of JS_GetPrivate() in native class methods that need the class instance's private data pointer and will do bad things if that pointer points to something other than what is expected. mcmlxxix (matt) discovered that using Object.apply(), you can invoke class methods where the 'this' instance is a different class. This would result in
"Internal Error: No Private Data." or a crash.
So now, gracefully detect this condition and report a meaningful error: "'<class-name>' instance: No Private Data or Class Mismatch"
Also, important to note: if the method uses JS_THIS_OBJECT to get the JSObject* to pass to JS_Get*Private, then it must do this *before* it calls JS_SET_RVAL.
From jsapi.h:
* NB: there is an anti-dependency between JS_CALLEE and JS_SET_RVAL: native
* methods that may inspect their callee must defer setting their return value
* until after any such possible inspection. Otherwise the return value will be
* inspected instead of the callee function object.
The js_crypt*.c files still need this treatment.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net