[Freeciv-Dev] [bug #19047] outdated comment in common/unit.c

2016-05-05 Thread Marko Lindqvist 
Update of bug #19047 (project freeciv):

  Status:  Ready For Test => Fixed  
 Assigned to:None => cazfi  
 Open/Closed:Open => Closed 


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

[Freeciv-Dev] [bug #19047] outdated comment in common/unit.c

2016-05-03 Thread Marko Lindqvist 
Update of bug #19047 (project freeciv):

  Status:None => Ready For Test 
 Planned Release:2.6.0, 3.0.0 => 2.5.4, 2.6.0, 3.0.0

___

Follow-up Comment #2:

I think it has referred to autosettler stuff that now lives in
server/advisors/autosettlers.c.
However, that code has also evolved so that it *does* use
can_unit_do_activity_targeted_at(), and that's not a "major CPU hog".

Attached patch just removes the comment.


(file #27293)
___

Additional Item Attachment:

File name: AutosettlerCommentRm.patch Size:0 KB


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

[Freeciv-Dev] [bug #19047] outdated comment in common/unit.c

2016-01-20 Thread Marko Lindqvist 
Update of bug #19047 (project freeciv):

Category:None => general
 Planned Release: => 2.6.0, 3.0.0   


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

[Freeciv-Dev] [bug #19047] outdated comment in common/unit.c

2012-03-11 Thread Jacob Nevins 
Follow-up Comment #1, bug #19047 (project freeciv):

I think the file (which was server/settlers.c when the comment was added,
r2373 http://svn.gna.org/viewcvs/freeciv?revision=2373view=revision in
2000) is now ai/aisettler.c on S2_3, ai/default/aisettler.c on S2_4.

However, I'm not sure exactly which code was being referred to and where that
might have ended up now. Perhaps it's referring to the stuff that's now in
server/advisors/infracache.c?

___

Reply to this item at:

  http://gna.org/bugs/?19047

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

[Freeciv-Dev] [bug #19047] outdated comment in common/unit.c

2011-11-23 Thread Michal Mazurek 

URL:
  http://gna.org/bugs/?19047

 Summary: outdated comment in common/unit.c
 Project: Freeciv
Submitted by: akfaew
Submitted on: Wed Nov 23 11:05:46 2011
Category: None
Severity: 3 - Normal
Priority: 5 - Normal
  Status: None
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 
 Discussion Lock: Any
Operating System: None
 Planned Release: 

___

Details:

A comment in common/unit.c says:

923   Note that if you make changes here you should also change the code
for
924   autosettlers in server/settler.c. The code there does not use this
925   function as it would be a major CPU hog.

There is no server/settler.c. This comment appears in S2_3 and trunk.




___

Reply to this item at:

  http://gna.org/bugs/?19047

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev