[Freeciv-Dev] [patch #1794] CALL_AI_FUNC macro

2010-08-05 Thread Marko Lindqvist

Update of patch #1794 (project freeciv):

  Status:  Ready For Test = Done   
 Open/Closed:Open = Closed 


___

Reply to this item at:

  http://gna.org/patch/?1794

___
  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] [patch #1794] CALL_AI_FUNC macro

2010-08-03 Thread Marko Lindqvist

Follow-up Comment #5, patch #1794 (project freeciv):

 This will not work for
 enum unit_move_result (*auto_explorer)(struct unit *punit);
 but this function will be moved to advisor, or? 

Yes, it's removed from AI interface in patch #1798 that this ticket depends
on.

___

Reply to this item at:

  http://gna.org/patch/?1794

___
  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] [patch #1794] CALL_AI_FUNC macro

2010-08-03 Thread Marko Lindqvist

Follow-up Comment #6, patch #1794 (project freeciv):

* Updated against svn
* Renamed macro CALL_AI_FUNC as CALL_PLR_AI_FUNC since we probably want to
introduce CALL_AI_FUNC macro that calls ai type functions without player
handle

(file #9676)
___

Additional Item Attachment:

File name: CallPlrAiFunc_1794-2.diff  Size:9 KB


___

Reply to this item at:

  http://gna.org/patch/?1794

___
  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] [patch #1794] CALL_AI_FUNC macro

2010-08-02 Thread Marko Lindqvist

Update of patch #1794 (project freeciv):

 Summary: unify check for ai functions = CALL_AI_FUNC macro 
   


___

Reply to this item at:

  http://gna.org/patch/?1794

___
  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] [patch #1794] CALL_AI_FUNC macro

2010-08-02 Thread Marko Lindqvist

Additional Item Attachment, patch #1794 (project freeciv):

File name: CallAiFunc_1794.diff   Size:9 KB


___

Reply to this item at:

  http://gna.org/patch/?1794

___
  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] [patch #1794] CALL_AI_FUNC macro

2010-08-02 Thread Marko Lindqvist

Update of patch #1794 (project freeciv):

  Status: In Progress = Ready For Test 


___

Reply to this item at:

  http://gna.org/patch/?1794

___
  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] [patch #1794] CALL_AI_FUNC macro

2010-08-02 Thread Matthias Pfafferodt

Follow-up Comment #4, patch #1794 (project freeciv):

 No. It should be possible with variable arguments. I'll give
 it a try. 

This will not work for 

enum unit_move_result (*auto_explorer)(struct unit *punit);

but this function will be moved to advisor, or?

 (btw. What are our current standards about assigning own
 patches to self? I noticed that you had taken this one
 already)

I did take my own patches to get an overview about them and after some time
started to apply them myself. It was wrong as some checking of the patches
was missing (especially the client side); I will stop.

___

Reply to this item at:

  http://gna.org/patch/?1794

___
  Nachricht geschickt von/durch Gna!
  http://gna.org/


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