Hi all, thanks for the feedback. Jon -- I will reach out to Mark and see what his plans are/were.
JP -- Thanks for the explanation Tom -- Thanks for the explanation. I agree it's a good idea to look into Cairo for printing. Yes, I should be able to fix the issue with PAINTER. -Jon On Wed, Feb 15, 2017 at 3:44 AM, Tomasz Wlostowski < [email protected]> wrote: > On 14.02.2017 19:38, Jon Evans wrote: > > Hi all, > > > > I want to get familiar with the GAL codebase, and it occurred to me that > > it might be fun to play with porting GerbView to GAL. I know it is on > > the 6.x roadmap, but it seemed to me that it would be mostly not > > dependent on any other changes that I see on the roadmap or have seen > > people talking about on the mailing list. > > > Hi Jon, > > Many thanks for your involvement. I think working on a GAL port of > GerbView is a great idea, even though it's not planned (yet) for the > nearest release. > > > - Is anyone currently working on this? > > I don't think so. > > > > > - If a GAL port is "feature-complete", is there any reason for the app > > to retain the legacy graphics code, or can it just provide OpenGL and > > Cairo backends? My impression is the only think keeping legacy canvas > > in pcbnew is feature differences between legacy and GAL, but I wanted to > > check if there are other reasons. > > Except for printing mentioned by JP, there's no reason to keep legacy > drawing code. I'm quite sure, though, printing could be handled by > GAL-Cairo with some minor modifications. > > > > > - In my first hour of looking at the code for this, it seems like the > > GAL code currently has some interdependence with pcbnew that needs to be > > straightened out before using GAL in other applications. For example, > > EDA_DRAW_PANEL_GAL depends on PCB_PAINTER, not a generic PAINTER. Is > > this an open problem for anyone to tackle, or does anyone actively have > > plans to refactor this? > > It's an obvious mistake, EDA_DRAW_PANEL_GAL should depend on the base > PAINTER class, and the PCB_PAINTER should be created in the > PCB_DRAW_PANEL_GAL. Would you be able to fix this? > > Cheers, > Tom > > >
_______________________________________________ Mailing list: https://launchpad.net/~kicad-developers Post to : [email protected] Unsubscribe : https://launchpad.net/~kicad-developers More help : https://help.launchpad.net/ListHelp
