Nathaniel Smith wrote: > On Mon, Jan 26, 2009 at 3:22 AM, Laurent Gautier <lgaut...@gmail.com> wrote: >> Nathaniel Smith wrote: >>> There are a number of callbacks in rinterface that one can set to >>> control R's console -- setWriteConsole, setReadConsole, etc. This >>> patch fixes to limitations with the current API: >> Thanks. >> Can you sent it again ? The patch seems to have gotten eaten on the way. > > Sigh. Sorry, here you go. > >> Indeed. I agree. >> I have been also considering putting all callbacks as attributes of a >> structure (since there are more callbacks in the dev version). Now I see >> that this could also solve the issue you are describing. > > True, that might be a prettier solution. It might be more tiresome to > code up at the C level, but then I find writing against the Python/C > api directly impossibly tiresome to start with ("once you Pyrex you > never go back"), so my impression isn't worth much :-).
I am thinking of applying your patch as submitted and make that structure at the Python level first, and see how the idea is. >> It is set to NULL at the C-level to cut corners (limited development time), >> and getting back to NULL should not be done (it might end in a segfault). >> The Python-level __init__.py in rinterface is immediately setting a value. > > The code I saw checked seemed to correctly check for NULL, but okay. I was too quick at thinking that my code would break. NULL is consistently checked for. >> It should build. Is that a checkout from the tip at bitbucket ? >> What platform are you on (OS, R version, Python version). > > Yes, from the tip, on Debian stable x86-64 with packaged python 2.5 > and own-built R 2.8.0. The curious thing is that it builds correctly, > and ldd says that it is linked to libR.so, but when imported it has > symbols it can't find. odd. > -- Nathaniel > ------------------------------------------------------------------------------ This SF.net email is sponsored by: SourcForge Community SourceForge wants to tell your story. http://p.sf.net/sfu/sf-spreadtheword _______________________________________________ rpy-list mailing list rpy-list@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/rpy-list