Hi Louis,

The common point of all of these 3 API (socketclient, sockserver, pipe) is this :

https://github.com/APE-Project/APE_Server/blob/master/modules/libape-spidermonkey.c#L2880

I guess I forget to add "JS_SET_RVAL" on the newly created object.

Try to add :

JS_SET_RVAL(cx, vpn, OBJECT_TO_JSVAL(obj));

at every place where JS_NewObjectForConstructor is used.

Let me know,

Anthony

Le 2013-02-04 02:49, Louis Charette a écrit :
I've been looking at the source code for the past two days. I can
confirm that MySQL support broke with the Spidermonkey update in
commits by Paraboul from June 25 2011 like Peter suggested somewhere
else [1].

I also found something interesting. If MySQL is not working, those
three feature are not working either:

* Sockclient: http://www.ape-project.org/docs/server/sockets/sockclient.html

* Sockserver: http://www.ape-project.org/docs/server/sockets/sockserver.html

        * Custom pipe: http://www.ape-project.org/docs/server/more/pipe.html

On my test server, none of this is working, including MySQL, on a
fresh build of the master branch from GitHub. I believe it's all
linked together. CAN SOMEONE CONFIRM THAT IF MYSQL IS NOT WORKING,
THOSE THREE FEATURE ARE NOT WORKING AS WELL?

I can't say why, but in libape_spidermonkey.c, around line 3053, the
JS_INITCLASS calls doesn't seems return an object. A function is
returned instead. This can be seen from Javascript like this:

var sql = new Ape.MySQL("ip:port", "user", "password", "database");
Ape.log(sql);

Before commit b89de0ed149aa4a5e2b539ff516da165ba607b26, this would be
printed in the Ape log:

[object MySQL]

After the commits, this is printed in the logs:

2013-02-02 10:44:07 - libape-spidermonkey.c:2475 - JavaScript :
function MySQL() {

 [native code]

}

Maybe something changed in spidermonkey from version 1.8.2 to 1.8.5.
I already asked on their group for more info. It's weird since
Spidermonkey documentation [2] state that JS_initClass should return
an object.

Unfortunately my C skills are limited and I can't do more for now.
I'll keep you posted if I find something else.

 - Louis

 --
 --
 You received this message because you are subscribed to the Google
 Groups "APE Project" group.
 To post to this group, send email to [email protected]
 To unsubscribe from this group, send email to
 [email protected]
 For more options, visit this group at
 http://groups.google.com/group/ape-project?hl=en [3]
 ---
 APE Project (Ajax Push Engine)
 Official website : http://www.ape-project.org/ [4]
 Git Hub : http://

unsubscribe from this group and stop receiving emails from it, send an email to [email protected].
For more options, visit
pt_out">https://groups.google.com/groups/opt_out.



Links:
------
[1] https://groups.google.com/forum/#!msg/ape-project/7JXRBFXf-SI/Trt8ApIXnwgJ
[2]

https://developer.mozilla.org/en-US/docs/SpiderMonkey/JSAPI_Reference/JS_InitClass
[3] http://groups.google.com/group/ape-project?hl=en
[4] http://www.ape-project.org/

--
--
You received this message because you are subscribed to the Google
Groups "APE Project" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/ape-project?hl=en
---
APE Project (Ajax Push Engine)
Official website : http://www.ape-project.org/
Git Hub : http://github.com/APE-Project/

--- You received this message because you are subscribed to the Google Groups "APE Project" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to