Well, the GTK version is not officially finished yet, and the TODO_GTK was
used as a kludge to mark all the code that still has to be converted from
Xaw to GTK. But there doesn't seem to be any reason for this particular
part not being done yet. Except perhaps that I did not understand the
original Xaw code (ManInner), for which ManProc is just a wrapper to
interface platform-dependent code to the new platform-independent way of
calling things from the 'middle-end'.
It seems that ManInner considers the possibility that it is called with
(non-NULL) parameters, which, even in the Xaw version, would never happen
when it was called because the user clicked the 'man' menu item. My guess
is that in the original XBoard is was possible to define a key binding
(external to XBoard, in the .xresources file) to ManProc(xxx), where xxx
could be the name of any man page, and not necessarily xboard, to summon
up that man page. I am not sure why anyone would ever want to do such a
thing.
If we drop that
> void ManProc ()
> { // called from menu
> system("xterm -e man xboard &", name);
> }
(and deleting ManInner) would do it in a straightforward way (but only for
'man xboard').
Op Zo, 24 november, 2013 3:43 pm schreef Joshua Pettus:
> I noticed that even on my linux install, xboard isnt even sending a call
> for xterm when man xboard is selected under the help menu. Info works
> just fine though. But something is not right with this passage under
> gtk/xboard.c:
>
>
> ManInner (Widget w, XEvent *event, String *prms, Cardinal *nprms)
> { // called as key binding
> char buf[MSG_SIZ]; String name;
> if (nprms && *nprms > 0) name = prms[0]; else name = "xboard";
snprintf(buf,
> sizeof(buf), "xterm -e man %s &", name); system(buf); }
> #endif
>
>
> void ManProc ()
> { // called from menu
> #ifdef TODO_GTK
> ManInner(NULL, NULL, NULL, NULL);
> #endif
> }
>
>
> Of course this works in Xaw version of xboard, so I wonder if the #ifdef
> TODO_GTK is where its getting hung up.
> _______________________________________________
> Bug-XBoard mailing list
> Bug-XBoard@gnu.org
> https://lists.gnu.org/mailman/listinfo/bug-xboard
>
>
_______________________________________________
Bug-XBoard mailing list
Bug-XBoard@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-xboard
