gEDA-dev: _Other_ code problems

[Peter Clifton]

On Mon, 2008-04-28 at 02:06 -0400, DJ Delorie wrote:
> I'm not going to try to apply and test these tonight; it's too late
> for me for that kind of stuff.  Comments I can do though..
> 
> > There's a typo in an assert() in rtree.c.  The fix is to compare Y with
> > Y, not X, in the last >:
> 
> OK.
> 
> > The batch HID won't even build, because it's missing file watch and
> > block hook functions, and its progress function return type is wrong.

Sorry, file watch and block hooks were things I added to the HID struct
when adding DBus support to the Lesstif and GTK HIDs.

Untested patch attached, which stubs these out.

> I use it for the web server, so it doesn't get rebuilt often.  I'll
> look into it unless Dan beats me to it.

It would be worth while checking this works OK... should not be any
issue if built without DBus support. Certainly DBus support won't work
right with these just being empty stubs.

-- 
Peter Clifton

Electrical Engineering Division,
Engineering Department,
University of Cambridge,
9, JJ Thomson Avenue,
Cambridge
CB3 0FA

Tel: +44 (0)7729 980173 - (No signal in the lab!)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: add_missingg_batch_stubs.diff
Type: text/x-patch
Size: 1101 bytes
Desc: not available
Url : http://www.seul.org/pipermail/geda-dev/attachments/20080428/1506a957/attachment.bin