From: Sam S. <sd...@gn...> - 2005-10-03 16:50:21
|
> * Tomas Zellerin <mryyreva@tznvy.pbz> [2005-10-03 08:30:26 +0200]: > >> >> >> >> does my module work for you? >> > >> > Mostly, but not here. Try add-defun something, and then either list >> > all known functions, or try to get the function using named-function. >> >> how do I list all known functions? > I do it by binding dump-functions to a key in inputrc and pressing the key. > "\ea": dump-functions cool! how do I bind F1? "\e[11~" (suggested in the manual) does not work. (I use cygwin/bash) >> > The former case is probably more instructive, as you can see that >> > there is new function with name containing strange characters - new >> > content of given place on stack, I guess. >> >> I guess :alloc :malloc-free is needed. >> > Yes, I did it that way and it worked. You will get a leak every time > you change binding to same string, but why someone should do it? this should be reflected in the (:documentation "string") of the call out. >> well, if we are at it, why not do everything? > There are reasons I see not to include everything : > - unused functions will not be tested and could cause crashes untested functions should be marked as such with a comment, e.g.: (def-call-out foo (:name "bar") ; untested ....) > - unused functions are not needed (there is lot of duplication through > the interface) :-) these are the famous last words: "I don't need it, so nobody does". let us not fall into this trap. people _will_ complain - just like you did. > - someone would have to write it, and I dont understand what some > functions do, in particular how completion works. > Of course, all can be disagreed (you wont get testing unless you include it). > > A style question - why are defined variables not starred? good question. we do not star foreign variables, see grep -r -i def-c-var modules/ I don't know why, it's just they way it always has been. >> Alas, the patch was corrupt (lines wrapped &c) >> the best way would be to open a "patch" tracker issue on SF. >> thanks. > It is there. thanks. so now you can update the patch! -- Sam Steingold (http://www.podval.org/~sds) running w2k <http://www.openvotingconsortium.org/> <http://truepeace.org> <http://pmw.org.il/> <http://www.dhimmi.com/> <http://www.mideasttruth.com/> We are born naked, wet, and hungry. Then things get worse. |