[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 <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/
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: layout.C in DDD sources,
Andreas Zeller <=