Peter Clifton wrote:
> On Tue, 2007-06-12 at 10:00 -0400, DJ Delorie wrote:
>>> If your busy (but approve the patch - less superfluous white-space
>>> changes), I could apply it to CVS when I get a moment during the
>>> week.
>> Go ahead. I was waiting to see what the opinion was on the two
>> different actions that did the same thing, but that seems to be common
>> anyway.
>
> Ok - I'd not noticed PasteBuffer(Save,...)
>
> For the list:
>
> The menus call the HID action Save(PasteBuffer) for the relevant item.
>
> In the GTK hid, this action then calls SaveAs(PasteBuffer,....).
>
> The lesstif HID uses:
>
> if (strcasecmp (function, "PasteBuffer") == 0)
> hid_actionl ("PasteBuffer", "Save", name, NULL);
>
>
> I get what you're saying now DJ, and see the concern. To ensure
> consensus here, Dan, Harry - how would you prefer to fix this?
I'll fix it by making the gtk HID's Save() action act more like the
lesstif one. In general I'd like to keep the actions in the gui's
acting the same if possible.
> Does anyone recall / know when the problem appeared?
"cvs blame" put the finger on me. Looks like when I converted the gtk
gui to the user defined menus, I switched from PasteBuffer(Save) to
Save(PasteBuffer) without really meaning to and the gtk Save() didn't
support PasteBuffer as an argument. I'll fix it.
-Dan
_______________________________________________
geda-dev mailing list
[email protected]
http://www.seul.org/cgi-bin/mailman/listinfo/geda-dev
