This is the mail archive of the xconq7@sources.redhat.com mailing list for the Xconq project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: code in adjust_tooling_crossover?


On Fri, 2002-07-12 at 15:15, Jim Kingdon wrote:
> Uh, I think you are looking for plan->maingoal->args[0] rather than
> unit2->type.

plan->maingoal->args[0]?  I'll try that.

> 
> Cleaning up the variable names in this function would be helpful.
> unit2 is only used in one small place (and so I'd move it there).
> Likewise for u2 (and unit2 and u2 don't have anything to do with each
> other :-().

I guess that explains why it looks so confusing.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]