[Freeciv-Dev] [patch #1671] Remove Bash syntax from Bourne shell scripts

2010-07-28 Thread Matthias Pfafferodt 

Update of patch #1671 (project freeciv):

  Status:None => Ready For Test 
 Assigned to:None => syntron
 Planned Release: => 2.2.3,2.3.0


___

Reply to this item at:

  

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


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

[Freeciv-Dev] [patch #1746] [players] send diplstate as separate network packages

2010-07-28 Thread Matthias Pfafferodt 

Update of patch #1746 (project freeciv):

Priority:   1 - Later => 5 - Normal 
  Status:   Postponed => Ready For Test 
 Assigned to:None => syntron

___

Follow-up Comment #5:

(some time later)

after a check of the possibility of this change it is done; I could not test
it without coding it ...

20100728-trunk-packet-player-diplstate.patch

* implements the new network packet

20100727-10-trunk-package-player_diplstate.patch
20100727-11-trunk-remove-old-diplstate-code.patch

* the first of this two patches can be used to compare both implementations;
the second patch removes the old one

results:

packet_player_info: down to approx. 300 byte
packet_player_diplstate: approx. 10 byte

(file #9604, file #9605, file #9606)
___

Additional Item Attachment:

File name: 20100727-10-trunk-package-player_diplstate.patch Size:15 KB
File name: 20100728-trunk-packet-player-diplstate.patch Size:17 KB
File name: 20100727-11-trunk-remove-old-diplstate-code.patch Size:7 KB


___

Reply to this item at:

  <http://gna.org/patch/?1746>

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


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

[Freeciv-Dev] [patch #1746] [players] send diplstate as separate network packages

2010-07-28 Thread Matthias Pfafferodt 

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

got some numbers:


2: got packet type=(PACKET_PLAYER_INFO)51 len=1548
2: packet_player_info_100: got info about (1)
2:   no old info
2:   got field 'name'
2:   got field 'username'
2:   got field 'city_style'
2:   got field 'nation'
2:   got field 'team'
2:   got field 'diplstates'
2:   got field 'researching'
2:   got field 'science_cost'
2:   got field 'tech_goal'
2:   got field 'revolution_finishes'
2:   got field 'ai_skill_level'
2:   got field 'inventions'
2:   got field 'love'


so 1550 bytes is way below 4096 bytes even if not all fields were send

___

Reply to this item at:

  

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


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

[Freeciv-Dev] [patch #1746] [players] send diplstate as separate network packages

2010-07-28 Thread Matthias Pfafferodt 

Follow-up Comment #3, patch #1746 (project freeciv):

Thinking a little bit about the memory requirements:

'struct diplstate' takes 28 byte on my system. It is further serialised by
dio_put_diplstate()/dio_get_diplstate() to (if I'm right) 9 byte. Thus, for
128 players 1152 bytes are needed.

'struct player' takes 2808 bytes; as network package it will be even less
(below 1000 bytes).

I do not have hard numbers but it seems to be OK to keep the diplstate as
part of the player package at the moment.

> player->real_embassy could also be moved into diplstate. Thus
> only one part of the player package is per (defined) player 

This is not a good idea. In the struct it will be one bool (= int = 4byte
(32bit) or 8byte(64bit)) for each player while as bitvector only 16bytes are
used for _all_ players.

___

Reply to this item at:

  

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


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

[Freeciv-Dev] [patch #1778] [interface] define the ai via the new interface

2010-07-28 Thread Matthias Pfafferodt 

Update of patch #1778 (project freeciv):

  Depends on: => patch #1732


___

Reply to this item at:

  

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


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

[Freeciv-Dev] [patch #1778] [interface] define the ai via the new interface

2010-07-28 Thread Matthias Pfafferodt 

Update of patch #1778 (project freeciv):

  Depends on: => patch #1772


___

Reply to this item at:

  

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


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

[Freeciv-Dev] [patch #1772] [interface] define an interface to call different functions at client and server

2010-07-28 Thread Matthias Pfafferodt 

Update of patch #1772 (project freeciv):

 Summary: [dbv] define an interface to call different
functions at client and server => [interface] define an interface to call
different functions at client and server


___

Reply to this item at:

  

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


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

[Freeciv-Dev] [patch #1773] [dbv][interface] remove tile:tile_seen

2010-07-28 Thread Matthias Pfafferodt 

Update of patch #1773 (project freeciv):

 Summary: [dbv] remove tile:tile_seen => [dbv][interface]
remove tile:tile_seen


___

Reply to this item at:

  

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


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

[Freeciv-Dev] [patch #1778] [interface] define the ai via the new interface

2010-07-28 Thread Matthias Pfafferodt 

URL:
  <http://gna.org/patch/?1778>

 Summary: [interface] define the ai via the new interface
 Project: Freeciv
Submitted by: syntron
Submitted on: Mittwoch 28.07.2010 um 13:13
Category: general
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: syntron
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.3.0

___

Details:

* remove ./common/ai.c as it is empty

* move call_incident() into a new file at ./server/advisors/advtools.c
together with amortize()?
* merge ai.h and into interface.h?



___

File Attachments:


---
Date: Mittwoch 28.07.2010 um 13:13  Name:
20100728-10-trunk-define-the-ai-via-the-new-interface.patch  Size: 25kB   By:
syntron

<http://gna.org/patch/download.php?file_id=9602>

___

Reply to this item at:

  <http://gna.org/patch/?1778>

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


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

[Freeciv-Dev] [patch #1773] [dbv] remove tile:tile_seen

2010-07-28 Thread Matthias Pfafferodt 

Follow-up Comment #1, patch #1773 (project freeciv):

- move client_map_known_and_seen into client_main.c
- fix some changes due to the updated interface

(file #9601)
___

Additional Item Attachment:

File name: 20100728-08-trunk-remove-tile-tile_seen.patch Size:19 KB


___

Reply to this item at:

  <http://gna.org/patch/?1773>

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


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

[Freeciv-Dev] [patch #1772] [dbv] define an interface to call different functions at client and server

2010-07-28 Thread Matthias Pfafferodt 

Follow-up Comment #1, patch #1772 (project freeciv):

cleanup the interface; now no extra functions are needed

(file #9600)
___

Additional Item Attachment:

File name:
20100728-07-trunk-define-an-interface-to-call-different-functions-at-c.patch
Size:7 KB


___

Reply to this item at:

  <http://gna.org/patch/?1772>

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


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

[Freeciv-Dev] [bug #16305] Incorrect sabotage handling for diplomats

2010-07-28 Thread Jacob Nevins 

Follow-up Comment #5, bug #16305 (project freeciv):

The Civ2 manual at civfanatics.com
 doesn't
indicate that Spies are more likely to succeed at anything than Diplomats,
except in the case of diplomat defence (which we already have). (Some of the
details in that manual sound different to our implementation, but there's not
enough detail in the manual to be absolutely sure.)

I don't think the Apolyton Great Library thread
 shows evidence
for such a difference either.

(I don't have Civ2 to try it myself.)

___

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 #16296] gtk client fails to sort files in load-save dialog

2010-07-28 Thread Jacob Nevins 

Follow-up Comment #5, bug #16296 (project freeciv):

On my system[*], where this has always appeared to more or less work
(presumably by luck with the specific sort implementation), I've confirmed
that the behaviour is unchanged with this patch -- I still get save files
sorted in reverse date order (good).

[*] Ubuntu Linux 9.04 amd64.

___

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] [patch #1421] Californian nation

2010-07-28 Thread J.M. Maalderink 

Update of patch #1421 (project freeciv):

  Status:  Ready For Test => Done   
 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