Re: AADL plugin

[Lars Clausen <lars raeder dk>]

On Wed, 2005-11-09 at 20:03 +0100, Lars Clausen wrote:
> On Tue, 2005-10-18 at 22:28 +0100, Pierre DUQUESNE wrote:
> > Hi list,
> >
> > I created a patch to Dia (CVS version) that add an AADL plugin i wrote.
> >
> > There is only 1 line added to the original source (check out the first
> > lines of the patch), everything else are new files. 
> >
> > I'd be glad to see it merged to the dia cvs tree.
>
> Thank you!  Looks like a fairly complete set to me (not knowing AADL:)
>
> A few comments:
>
> I'm not too happy about the line you've added to modify_tool.c, it looks
> like a random kludge - I can't tell what it's supposed to do and it
> doesn't seem to generalize well, once I understand it I'll see if
> there's a more generic way to do the same thing.

Is it just because you're not done with them that the HANDLE_CUSTOM9
type is not actually used in your program?  I don't see anyplace where
it appears to be used in a way similar to what you described earlier.

-Lars

-- 
Lars Clausen <lars raeder dk>