Re: layout.C in DDD sources

[Andreas Zeller]

Hi!

Emmanuel Briot <briot@gnat.com> writes:

> I was looking at DDD's sources this week-end, and saw a strange
> function in layout.C.  I am not sure this file is used anymore, but
> even though you might want to fix it:
> 
> 
> int Layout::graphNewNodeID()
> {
>     int counter = 1000 ;
> 
>     return counter++ ;
> }
> 
> 
> 
> Should probably be:
> 
> 
> int Layout::graphNewNodeID()
> {
>     static int counter = 1000 ;
> 
>     return counter++ ;
> }

I have applied your patch.  I'm not sure either what this function is
for, but your patch seems most reasonable :-)

Andreas

-- 
Andreas Zeller     Universitaet Passau
                   http://www.fmi.uni-passau.de/st/staff/zeller/