Peter TB Brett writes: > [...] > > Yes, I figured that. BTW the 'Shift :' is impossible to realize on my > > keyboard. This is a problem that is likely to happen on other > > combinations but it is the first time I am hit by that. > > What about binding it to just ';'?
Well it was more of a remark than request to change it. I am not sure we can find a shortcut that will be ok with every keyboard layout. So ';' is likely to be a problem for someone else and we only moved it. > [...] > > Good but avoid adding guile code to i_callback_*(): keep the > > separation between guile code and callbacks. > > > > So in your example the rest arguments "opamp.sym" has to be analyzed > > by g_keys_add_component() (the function is customized and not provided > > by the DEFINE_G_KEY macro) that pass it to i_callback_add_component() > > as a C string (again its prototype is customized). > > >From what you're saying it sounds like you think I should revert commit > 61255c9b68aeae2ae845eea82babf445036a72f1. Is that correct? Partially. What I think should be reverted here is the addition of a 'SCM rest' parameter to functions in i_callbacks.c. Regards, Patrick _______________________________________________ geda-dev mailing list [email protected] http://www.seul.org/cgi-bin/mailman/listinfo/geda-dev
