bug-ddd
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: layout.C in DDD sources


From: Andreas Zeller
Subject: Re: layout.C in DDD sources
Date: 26 Mar 2001 17:41:53 +0200
User-agent: Gnus/5.0808 (Gnus v5.8.8) XEmacs/21.1 (Cuyahoga Valley)

Hi!

Emmanuel Briot <address@hidden> 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/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]