gEDA-dev: o_redraw_single() has moved to gschem

[Steve Meier]

I also think that objects in libgeda should know which application
function to call in order to draw them selves.

This allows the application to ask the page to draw itself and the page
to ask the complex components, segments, arcs, boxes, text etc to draw
themselves. Instead of the application having to go through the page and
get each item and then do the drawing.

When I say an object should draw itself I mean that the object should
call the appropriatly registered application function. In my view this
way if the library changes the application won't get broken.

Steve Meier


Bernd Jendrissek wrote:
> On 10/31/07, Peter Clifton <pcjc2 at cam.ac.uk> wrote:
>   
>> The move was made to push all drawing code and related variables into
>> gschem (where they arguably belong), o_redraw_single actually had very
>> little use in libgeda.
>>     
>
> Perhaps every OBJECT should have a draw() selector, and have gschem
> provide an OBJECT factory so it can bind its method to the selector?
> Just thinking aloud here...
>
>   
>> My offer to fix up anyone's outstanding patches
>> still stands, however I don't have time to do it at the moment.
>>     
>
> That's okay, I have time.
>
>   
>> ->CHANGED = 1; (off the top of my head) I think relates to the dirty
>> state of pages, so the program will complain about needing to save if
>> you try to quit.
>>     
>
> Damn, so not that either.
>
>   
>> I think the best option for now, (if applicable) is to move the code to
>> redraw after your renumber operation into gschem (or the whole lot).
>>     
>
> Yuck.  No, I'd really rather keep s_slot_link() in libgeda where it belongs.
>
> I think I'll hack up a GList-based Observer to tell gschem that the
> text has changed.  I'll have to register gschem with every TEXT that
> it draws and get it to re-do the prim_objs list according to the new
> text_string.
>
>
> _______________________________________________
> geda-dev mailing list
> geda-dev at moria.seul.org
> http://www.seul.org/cgi-bin/mailman/listinfo/geda-dev
>
>