[Freeciv-Dev] [patch #2776] Hunnic nation

2011-07-11 Thread Andrzej M. Gorzym

Follow-up Comment #2, patch #2776 (project freeciv):

I thought the most convincing is the hypothesis about the Turkish origin of
the Huns. A little later the Bulgarian tribes are considered descendants of
the Huns. In addition, Chuvash language, having supposedly come from the
Bulgarian dialects, has a very primitive characteristics compared to other
Turkic languages. Hence, it is believed that he could come from the language
of the Huns, because the next wave of expansion of Turkish use the more modern
languages. However, very possible that the original language of the Huns was a
different language, perhaps Yeniseian, but they could be Turkization still
somewhere in Asia.

For the names of archaeological sites related to the Huns, I'm still in the
course of their search.

___

Reply to this item at:

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

___
  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 #18323] Governments menu grows without bound, making it difficult/impossible to change government

2011-07-11 Thread Jacob Nevins

Update of bug #18323 (project freeciv):

  Status:  Ready For Test = Fixed  
 Open/Closed:Open = Closed 
 Summary: Cannot have any government but despots =
Governments menu grows without bound, making it difficult/impossible to change
government


___

Reply to this item at:

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

___
  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 #2776] Hunnic nation

2011-07-11 Thread J.M. Maalderink

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

There's no reason why Chuvash couldn't descend from any other old Turkish
language. It's well established that there were Turkic peoples in the region
less than a century after the Huns, so that would explain for the supposedly
archaic characteristics of Chuvash.

From what I know about the claim the Huns are Turkic is that it is mostly
based on etymologies for Hun personal names, and a lot of those claims are
rather spurious. I mean, basing Atilla on the Turkish word for 'father' as is
often claimed is rather silly, the word for father is something like atta or
papa in every human language, for the same reason mother is mama or nana, it's
just the first sounds babies make. Furthermore, one can just as easily make up
Uralic or Iranian etymologies for Hun names.

From what I remember from my medieval history classes, ethnic designations in
this era are notoriously fuzzy anyway. To begin with, most barbaric empires or
kingdoms were political rather than ethnic anyway, so it's likely the Huns
were multi-ethnic. Plus several peoples using the same name doesn't mean they
are the same people; many Roman and Byzantine authors were too lazy to
distinguish between ethnic groups so they just called everything that came out
of the steppes 'Scythians' or 'Huns' or whatever. Also it often happened that
a nation adopted the name of an ancient people they don't have any connection
with for prestige reasons.

___

Reply to this item at:

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

___
  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 #2780] Xiongnu nation

2011-07-11 Thread J.M. Maalderink

Update of patch #2780 (project freeciv):

  Status:None = In Progress
 Assigned to:None = mixcoatl   


___

Reply to this item at:

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

___
  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 #2773] Backtrace log callback for log levels = error

2011-07-11 Thread Marko Lindqvist

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

- Configure check for execinfo.h
- Fixed compiler warnings when backtrace functionality not available 

(file #13508)
___

Additional Item Attachment:

File name: RuntimeBacktrace_2773-2.diff   Size:7 KB


___

Reply to this item at:

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

___
  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 #2781] Anhaltian nation

2011-07-11 Thread J.M. Maalderink

URL:
  http://gna.org/patch/?2781

 Summary: Anhaltian nation
 Project: Freeciv
Submitted by: mixcoatl
Submitted on: Mon 11 Jul 2011 03:48:16 PM GMT
Category: rulesets
Priority: 5 - Normal
  Status: In Progress
 Privacy: Public
 Assigned to: mixcoatl
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 

___

Details:

Anhalt

flag by myself



___

File Attachments:


---
Date: Mon 11 Jul 2011 03:48:16 PM GMT  Name: anhaltian.ruleset  Size: 2kB  
By: mixcoatl

http://gna.org/patch/download.php?file_id=13509
---
Date: Mon 11 Jul 2011 03:48:16 PM GMT  Name: anhalt.svg  Size: 262B   By:
mixcoatl

http://gna.org/patch/download.php?file_id=13510

___

Reply to this item at:

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

___
  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 #2782] Tahitian (French Polynesian) nation

2011-07-11 Thread J.M. Maalderink

URL:
  http://gna.org/patch/?2782

 Summary: Tahitian (French Polynesian) nation
 Project: Freeciv
Submitted by: mixcoatl
Submitted on: Mon 11 Jul 2011 03:51:41 PM GMT
Category: rulesets
Priority: 5 - Normal
  Status: In Progress
 Privacy: Public
 Assigned to: mixcoatl
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 

___

Details:

Tahiti/French Polynesia

Flag adapted from Tobias Jakobs, Open Clip Art:
http://www.openclipart.org/detail/17757/flag-of-french-polynesia-by-tobias 



___

File Attachments:


---
Date: Mon 11 Jul 2011 03:51:41 PM GMT  Name: french_polynesia.svg  Size: 18kB
  By: mixcoatl

http://gna.org/patch/download.php?file_id=13511
---
Date: Mon 11 Jul 2011 03:51:41 PM GMT  Name: tahiti.ruleset  Size: 2kB   By:
mixcoatl

http://gna.org/patch/download.php?file_id=13512

___

Reply to this item at:

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

___
  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 #2783] African nation

2011-07-11 Thread J.M. Maalderink

URL:
  http://gna.org/patch/?2783

 Summary: African nation
 Project: Freeciv
Submitted by: mixcoatl
Submitted on: Mon 11 Jul 2011 03:56:00 PM GMT
Category: rulesets
Priority: 5 - Normal
  Status: In Progress
 Privacy: Public
 Assigned to: mixcoatl
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 

___

Details:

Africa

The African Union flag is poorly visible in a small size, and I couldn't find
a svg version of it under a free licence. Another option would be the
Pan-African flag but as far as I know that one is more associated with the
African diaspora then with Africa itself. So I made a flag with a silhouette
of Africa on a green background, with the shape of Africa taken from
http://commons.wikimedia.org/wiki/File:Continents.svg 



___

File Attachments:


---
Date: Mon 11 Jul 2011 03:56:00 PM GMT  Name: africa.ruleset  Size: 2kB   By:
mixcoatl

http://gna.org/patch/download.php?file_id=13513
---
Date: Mon 11 Jul 2011 03:56:00 PM GMT  Name: africa.svg  Size: 15kB   By:
mixcoatl

http://gna.org/patch/download.php?file_id=13514

___

Reply to this item at:

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

___
  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 #2738] Client - metaserver communication thread

2011-07-11 Thread Marko Lindqvist

Update of patch #2738 (project freeciv):

 Planned Release:   2.4.0 = 2.5.0  

___

Follow-up Comment #3:

This cannot work with dummy thread implementation. There is no other need for
real thread implementation at the moment, and I don't want make them hard
requirement for 2.4 just for this. This is not yet so important. 2.4 is first
version where threads are used at all, so let's have disabling threads as a
workaround for any threads related problems users encounter.

___

Reply to this item at:

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

___
  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 #2380] Client - metaserver communication with curl

2011-07-11 Thread Marko Lindqvist

Update of patch #2380 (project freeciv):

 Planned Release:   2.4.0 = 2.5.0  


___

Reply to this item at:

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

___
  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 #1845] [metaticket] Use curl for fetching files and pages over network

2011-07-11 Thread Marko Lindqvist

Update of patch #1845 (project freeciv):

 Planned Release:   2.4.0 = 2.5.0  


___

Reply to this item at:

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

___
  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 #2784] Somaliland nation

2011-07-11 Thread J.M. Maalderink

URL:
  http://gna.org/patch/?2784

 Summary: Somaliland nation
 Project: Freeciv
Submitted by: mixcoatl
Submitted on: Mon 11 Jul 2011 04:21:53 PM GMT
Category: rulesets
Priority: 5 - Normal
  Status: In Progress
 Privacy: Public
 Assigned to: mixcoatl
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 

___

Details:

Somaliland

Flag by myself, based on Saudi flag



___

File Attachments:


---
Date: Mon 11 Jul 2011 04:21:53 PM GMT  Name: somaliland.ruleset  Size: 2kB  
By: mixcoatl

http://gna.org/patch/download.php?file_id=13515
---
Date: Mon 11 Jul 2011 04:21:53 PM GMT  Name: somaliland.svg  Size: 30kB   By:
mixcoatl

http://gna.org/patch/download.php?file_id=13516

___

Reply to this item at:

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

___
  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 #18328] Ruleset server settings are case-sensitive

2011-07-11 Thread Micke

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

 Summary: Ruleset server settings are case-sensitive
 Project: Freeciv
Submitted by: someswede
Submitted on: Mon Jul 11 16:46:39 2011
Category: rulesets
Severity: 3 - Normal
Priority: 5 - Normal
  Status: None
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: SVN 19927
 Discussion Lock: Any
Operating System: None
 Planned Release: 

___

Details:

When setting server options in game.ruleset the values are case-sensitive;
for instance, having this gives an error:
 mapsize, player, FALSE
Can't read value for setting 'mapsize': In secfile_lookup_enum_data()
[registry.c:2499]: secfile 'data/nomodern/game.ruleset' in section 'settings':
Entry set0.value: no match for player.

whereas this works fine:
 mapsize, PLAYER, FALSE


However, when using the /set server command, it is not case-sensitive:
mikaels: 'set mapsize FULLSIZE'
Console: 'mapsize' has been set to Number of tiles (FULLSIZE).
mikaels: 'set mapsize PlAyeR'
Console: 'mapsize' has been set to Tiles per player (PLAYER).
mikaels: 'set mapsize fUllsIzE'
Console: 'mapsize' has been set to Number of tiles (FULLSIZE).
mikaels: 'set mapsize plaYER'
Console: 'mapsize' has been set to Tiles per player (PLAYER).




___

Reply to this item at:

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

___
  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 #2754] improved Roman nation

2011-07-11 Thread J.M. Maalderink

Update of patch #2754 (project freeciv):

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


___

Reply to this item at:

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

___
  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 #18328] Ruleset server settings are case-sensitive

2011-07-11 Thread Micke

Follow-up Comment #1, bug #18328 (project freeciv):

something, DISABLED, FALSE
does not work either, instead one has to use
something, 0, FALSE
even if the server command is /set something disabled

I'm not sure if this should be considered a bug or just an annoying
inconsistency.


___

Reply to this item at:

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

___
  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 #18329] Packet receiving code assumes sizeof(enum packet_type) == sizeof(int)

2011-07-11 Thread Marko Lindqvist

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

 Summary: Packet receiving code assumes sizeof(enum
packet_type) == sizeof(int)
 Project: Freeciv
Submitted by: cazfi
Submitted on: Mon 11 Jul 2011 08:45:29 PM EEST
Category: general
Severity: 3 - Normal
Priority: 5 - Normal
  Status: Ready For Test
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 
 Discussion Lock: Any
Operating System: None
 Planned Release: 2.3.0, 2.4.0

___

Details:

Packet receiving code depends on sizeof(enum packet_type) == sizeof(int).
Fix attached.




___

File Attachments:


---
Date: Mon 11 Jul 2011 08:45:29 PM EEST  Name: NetworkEnum.diff  Size: 792B  
By: cazfi

http://gna.org/bugs/download.php?file_id=13517
---
Date: Mon 11 Jul 2011 08:45:29 PM EEST  Name: NetworkEnum-S2_2.diff  Size:
794B   By: cazfi

http://gna.org/bugs/download.php?file_id=13518

___

Reply to this item at:

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

___
  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 #18329] Packet receiving code assumes sizeof(enum packet_type) == sizeof(int)

2011-07-11 Thread Marko Lindqvist

Update of bug #18329 (project freeciv):

 Planned Release:2.3.0, 2.4.0 = 2.2.7, 2.3.0, 2.4.0


___

Reply to this item at:

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

___
  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 #18331] Available themes are not listed alphabetically

2011-07-11 Thread Micke

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

 Summary: Available themes are not listed alphabetically
 Project: Freeciv
Submitted by: someswede
Submitted on: Mon Jul 11 18:11:57 2011
Category: client-gtk-2.0
Severity: 2 - Minor
Priority: 5 - Normal
  Status: None
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 19927
 Discussion Lock: Any
Operating System: GNU/Linux
 Planned Release: 

___

Details:

Menubar  Game  Options  Local client  Graphics: Theme
has a drop-down list; the available themes are not listed alphabetically.

http://i.imgur.com/HsKS0.png





___

Reply to this item at:

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

___
  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 #2786] Use gtk2 instead of plain gtk as example value for --enable-client

2011-07-11 Thread Marko Lindqvist

URL:
  http://gna.org/patch/?2786

 Summary: Use gtk2 instead of plain gtk as example value
for --enable-client
 Project: Freeciv
Submitted by: cazfi
Submitted on: Mon 11 Jul 2011 09:59:43 PM EEST
Category: docs
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.3.0, 2.4.0

___

Details:

Attached patch changes --enable-client documentation to refer to value gtk2
instead of gtk



___

File Attachments:


---
Date: Mon 11 Jul 2011 09:59:43 PM EEST  Name: GtkIsGtk2.diff  Size: 1kB   By:
cazfi

http://gna.org/patch/download.php?file_id=13521
---
Date: Mon 11 Jul 2011 09:59:43 PM EEST  Name: GtkIsGtk2-S2_3.diff  Size: 1kB 
 By: cazfi

http://gna.org/patch/download.php?file_id=13522

___

Reply to this item at:

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

___
  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 #2787] Windows icon for freeciv-modpack.exe

2011-07-11 Thread Marko Lindqvist

URL:
  http://gna.org/patch/?2787

 Summary: Windows icon for freeciv-modpack.exe
 Project: Freeciv
Submitted by: cazfi
Submitted on: Mon 11 Jul 2011 11:03:39 PM EEST
Category: module installer
Priority: 5 - Normal
  Status: In Progress
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.3.0, 2.4.0

___

Details:

freeciv-modpack.exe should have icon in Windows. Attached untested patch adds
that, except that actual gfx is missing.



___

File Attachments:


---
Date: Mon 11 Jul 2011 11:03:39 PM EEST  Name: MPIcon.diff  Size: 2kB   By:
cazfi

http://gna.org/patch/download.php?file_id=13523

___

Reply to this item at:

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

___
  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 #2788] Automatically prepend fc_ai_ to module names given for -L

2011-07-11 Thread Marko Lindqvist

URL:
  http://gna.org/patch/?2788

 Summary: Automatically prepend fc_ai_ to module names
given for -L
 Project: Freeciv
Submitted by: cazfi
Submitted on: Mon 11 Jul 2011 11:41:50 PM EEST
Category: ai
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.4.0

___

Details:

This patch makes freeciv to automatically add prefix fc_ai_ to ai module
names given for commandline option -L so one can (and should) omit it.




___

File Attachments:


---
Date: Mon 11 Jul 2011 11:41:50 PM EEST  Name: AutoModPrefix.diff  Size: 2kB  
By: cazfi

http://gna.org/patch/download.php?file_id=13524

___

Reply to this item at:

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

___
  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 #18333] assertion 'ai-phase_initilized' failed

2011-07-11 Thread Micke

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

 Summary: assertion 'ai-phase_initilized' failed
 Project: Freeciv
Submitted by: someswede
Submitted on: Mon Jul 11 21:09:29 2011
Category: None
Severity: 3 - Normal
Priority: 5 - Normal
  Status: None
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 19927
 Discussion Lock: Any
Operating System: GNU/Linux
 Planned Release: 

___

Details:

I'm guessing this is related to bugs 17728 and 18095 (for the 2.3.0 betas),
which are both marked as closed.

Upon turn 5 in my first game with this revision, I got 3 of these (with 6 AI
players):
[22:54:22] in ai_plr_data_get() [aidata.c::310]: assertion
'ai-phase_initialized' failed.
[22:54:22] Please report this message at http://gna.org/projects/freeciv/





___

Reply to this item at:

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

___
  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 #18333] assertion 'ai-phase_initialized' failed

2011-07-11 Thread Jacob Nevins

Update of bug #18333 (project freeciv):

 Release:   19927 = trunk r19927   
 Planned Release: = 2.4.0  
 Summary: assertion 'ai-phase_initilized' failed =
assertion 'ai-phase_initialized' failed

___

Follow-up Comment #1:

It sounds like you're talking about trunk (as there's no file/function named
like that in S2_3).

Bug #18095 was closed by suppressing the assertion failure on S2_3 only
(trunk was unchanged). The failure there was not reproducible on trunk.

I had a quick poke at this revision and didn't see anything of the kind after
29 turns with the default ruleset and aifill 7. Can you provide a savegame
from which it can be reproduced? Failing that, can you get a backtrace with
-F?

___

Reply to this item at:

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

___
  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 #2761] Add missing stdarg.h includes

2011-07-11 Thread Marko Lindqvist

Update of patch #2761 (project freeciv):

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


___

Reply to this item at:

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

___
  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 #18087] server segfault on a map with 1.3M tiles

2011-07-11 Thread Jacob Nevins

Follow-up Comment #14, bug #18087 (project freeciv):

 Oh there is so much stack abuse in freeciv.
 ...question: is allocating huge arrays on the stack instead of 
 allocating the same arrays on the heap (in the absence of 
 recursion etc) really an abuse?
Having read around a bit more, I'm now convinced it's not a good idea.
Wikipedia http://en.wikipedia.org/wiki/Stack-based_memory_allocation
suggests that a thread's stack size can be as small as a few dozen
kilobytes, and more specifically, the Windows documentation
http://msdn.microsoft.com/en-us/library/ms686774%28v=vs.85%29.aspx says The
default stack reservation size used by the linker is 1 MB.

What I think I'd missed with my it's all memory argument is that when
memory runs out, malloc() can return NULL (from which the application can
theoretically recover), whereas allocating memory on the stack can't fail, so
the OS has little option but to kill the entire app. It looks like as a result
of that, some (all?) OSes (such as Windows, see previous article) set aside
memory for app stacks, so that apps aren't killed for the crime of calling a
function when things get tight. Thus stack memory is more reliable than heap
memory, thus more precious, thus OSes place limits on it, thus it's not a good
idea to try to allocate arbitrarily large chunks of it.

Sorry if that was obvious to everyone else. Conclusion is, we shouldn't
allocate O(xsize*ysize) memory on the stack. IMO we should take the
stack-abuse changes on S2_3, as well as the assign_continent_flood() change
that we know we need. I've had a look and they look basically fine to me; more
in future comments.

___

Reply to this item at:

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

___
  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 #2789] 16bit packet type

2011-07-11 Thread Marko Lindqvist

URL:
  http://gna.org/patch/?2789

 Summary: 16bit packet type
 Project: Freeciv
Submitted by: cazfi
Submitted on: Tue 12 Jul 2011 01:16:20 AM EEST
Category: general
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.5.0

___

Details:

We discussed already when editor was in development that we may eventually
run out of unique packet numbers. We still have some space left, and there's
many ways to reuse packet numbers, but just to avoid any uglier solutions I'd
like to increase packet number space before this becomes a real problem.
Attached patch makes packet type 16 bit value (currently 8 bit). This is still
not very much tested, and anyway I'd like to commit it from the beginning of
the new cycle (soon after S2_4 branching) rather than
late in cycle that has used 8 bit types so far.




___

File Attachments:


---
Date: Tue 12 Jul 2011 01:16:20 AM EEST  Name: 16BitPacketType.diff  Size: 5kB
  By: cazfi

http://gna.org/patch/download.php?file_id=13525

___

Reply to this item at:

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

___
  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 #2790] Fix --enable-fcdb help default value

2011-07-11 Thread Marko Lindqvist

URL:
  http://gna.org/patch/?2790

 Summary: Fix --enable-fcdb help default value
 Project: Freeciv
Submitted by: cazfi
Submitted on: Tue 12 Jul 2011 01:23:05 AM EEST
Category: general
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.4.0

___

Details:

Help for --enable-fcdb claims that default is all when implementation is
actually no.

In addition to correcting the help, attached patch:
- Uses same handling for default value than explicitly set no
- Adds check for illegal value
 



___

File Attachments:


---
Date: Tue 12 Jul 2011 01:23:05 AM EEST  Name: FcdbConfDocFix.diff  Size: 825B
  By: cazfi

http://gna.org/patch/download.php?file_id=13526

___

Reply to this item at:

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

___
  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 #2791] CPUHog

2011-07-11 Thread Marko Lindqvist

URL:
  http://gna.org/patch/?2791

 Summary: CPUHog
 Project: Freeciv
Submitted by: cazfi
Submitted on: Tue 12 Jul 2011 01:30:06 AM EEST
Category: general
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.4.0

___

Details:

To save some CPU-cycles AI does not always calculate everything but uses good
enough existing values from different caches etc. I often find AI's bad
performance much worse problem than running out of CPU-cycles.
This patch adds some stuff to implement CPU-hog AI that hopefully performs a
bit better at the cost of using much more computing resources. In this initial
version only max number of cm-iterations is increased for CPU-hogs.
This patch makes it only easier to make CPU-hog version of freeciv. One
cannot change ai behavior runtime, but it still requires code change. CPU-hog
framework just makes that required code change one-liner. See comments in
player.h for full rationale.




___

File Attachments:


---
Date: Tue 12 Jul 2011 01:30:07 AM EEST  Name: CPUHog.diff  Size: 2kB   By:
cazfi

http://gna.org/patch/download.php?file_id=13527

___

Reply to this item at:

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

___
  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 #2792] Devel Stable URLs for freeciv-modpack

2011-07-11 Thread Marko Lindqvist

URL:
  http://gna.org/patch/?2792

 Summary: Devel  Stable URLs for freeciv-modpack
 Project: Freeciv
Submitted by: cazfi
Submitted on: Tue 12 Jul 2011 01:45:17 AM EEST
Category: module installer
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.4.0

___

Details:

Attached patch makes freeciv-modpack compilation to automatically to switch
default URLs to gna.org ones for stable releases. Development versions use
cazfi.net ones.




___

File Attachments:


---
Date: Tue 12 Jul 2011 01:45:17 AM EEST  Name: MPDevelStableURLs.diff  Size:
898B   By: cazfi

http://gna.org/patch/download.php?file_id=13528

___

Reply to this item at:

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

___
  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 #2427] [metaticket] Add missing function headers

2011-07-11 Thread Marko Lindqvist

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

gui-stub seems to be ok, and thus also gui-qt that started as copy of stub.
gui-gtk2 was finished before it was copied as base for gui-gtk3.

I don't think we need to bother with gui-ftwl and gui-win32 en route to
removal until someone starts saving them.

This leaves gui-sdl and gui-xaw, as well as server/scripting.

___

Reply to this item at:

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

___
  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 #18087] server segfault on a map with 1.3M tiles

2011-07-11 Thread Jacob Nevins

Follow-up Comment #15, bug #18087 (project freeciv):

Attached my version of file #12960, split into three separate patches and
tweaked a little.

The three patches:
* *trunk-S2_3-hugemap-stack-overflow.diff*: The stack overflow fixes. This is
what I propose we apply to S2_3 pre RC1, to deal with bug #17962. _I will do
so if there are no objections in 36h._
** I've reviewed this to check that no memory leaks came in with the
introduction of heap allocation due to returning without freeing. I only
noticed one: in sg_load_map_known()), SAVE_MAP_CHAR() - sg_failure_ret() -
sg_check_ret() can return prematurely. This is a leaky pattern and should be
dealt with (raise a ticket), but it's an error condition so I think we can
live without a fix for RC1.
** I've only reviewed the existing changes for correctness; I haven't looked
to see if there are any map-sized stack allocations that have been missed.
** I haven't reviewed the new flooding algorithm in detail. It looks vaguely
plausible.
** As a sanity check, I've played a single-player game with this patch (S2_3)
for a few tens of turns without the world ending. No autogame testing yet.
* *trunk-S2_3-hugemap-colatitude.diff*: This messes with colatitude stuff I
don't fully understand, so it seems riskier to me. Unless someone thinks
otherwise, I suggest we leave it out for 2.3.0-RC1, since it's not necessary
to fix the stack issues. Could add it back to a later 2.3.x.
** This bit of the patch is Matthias; it came in between file #12942 (comment
#4) and file #12947 (comment #5).
** Matthias: do you think this patch is useful/necessary for the map size
limits we have on S2_3?
* *trunk-S2_3-hugemap-biggermaps.diff*: This actually increases the maximum
map sizes. I suggest we leave this for trunk only.
** Aside: I think the correct way to express the relationship between
MAP_MAX_SIZE and MAX_DBV_LENGTH is to publish the latter in bitvector.h and
have a static (compile-time) assert in the map code. But we don't seem to have
a static assert mechanism yet. I may look into it.

My tweaks were:
* Remove memset()s that were redundant due to use of fc_calloc()
* Change free() to FC_FREE() throughout, for form's sake
* Add 'const' to constant arrays in smooth_int_map()
* Rationalise away 'ret' in rand_map_pos_filtered()

(file #13529, file #13530, file #13531)
___

Additional Item Attachment:

File name: trunk-S2_3-hugemap-stack-overflow.diff Size:16 KB
File name: trunk-S2_3-hugemap-colatitude.diff Size:1 KB
File name: trunk-S2_3-hugemap-biggermaps.diff Size:2 KB


___

Reply to this item at:

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

___
  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 #17962] Inability to generate/load huge maps on Windows

2011-07-11 Thread Jacob Nevins

Follow-up Comment #14, bug #17962 (project freeciv):

I intend to apply file #13531 (from bug #18087) to try to deal with this
before 2.3.0-RC1 is released.

Since there isn't much time before RC1, pre-commit testing would be
appreciated (especially by Windows users).

___

Reply to this item at:

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

___
  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 #17962] Inability to generate/load huge maps on Windows

2011-07-11 Thread Jacob Nevins

Update of bug #17962 (project freeciv):

  Status:None = Ready For Test 
 Assigned to:None = jtn


___

Reply to this item at:

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

___
  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 #18283] Comments about map size limits in map.h do not match code

2011-07-11 Thread Jacob Nevins

Update of bug #18283 (project freeciv):

  Status:   Need Info = Duplicate  
 Open/Closed:Open = Closed 


___

Reply to this item at:

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

___
  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 #18334] few dozen typos in 2.2.5

2011-07-11 Thread Karl Goetz

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

 Summary: few dozen typos in 2.2.5
 Project: Freeciv
Submitted by: kgoetz
Submitted on: Tue 12 Jul 2011 08:59:42 AM CST
Category: None
Severity: 2 - Minor
Priority: 5 - Normal
  Status: None
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 
 Discussion Lock: Any
Operating System: GNU/Linux
 Planned Release: 

___

Details:

Hi guys,
While updating the Debian packaging i came across a number of typos in the
game - it appears to build and run still, so could you please test and apply?
thanks,
kk



___

File Attachments:


---
Date: Tue 12 Jul 2011 08:59:42 AM CST  Name: trivial-typos.diff  Size: 6kB  
By: kgoetz

http://gna.org/bugs/download.php?file_id=13532

___

Reply to this item at:

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

___
  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 #18335] please change the colour of the research tab.

2011-07-11 Thread Karl Goetz

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

 Summary: please change the colour of the research tab.
 Project: Freeciv
Submitted by: kgoetz
Submitted on: Tue 12 Jul 2011 09:18:09 AM CST
Category: client-gtk-2.0
Severity: 3 - Normal
Priority: 5 - Normal
  Status: None
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 2.2.5
 Discussion Lock: Any
Operating System: GNU/Linux
 Planned Release: 

___

Details:

Hi,
Forwarding this from the debian bts, where i filed it origonally.


When a technology is successfully researched, freeciv turns the 'Research'
tab red. It would be good if it could use blue (like it does for the messages
box) as red is quite hard to see on the background window.




___

Reply to this item at:

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

___
  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 #2778] City map iterating macros that don't set _index

2011-07-11 Thread Marko Lindqvist

Update of patch #2778 (project freeciv):

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


___

Reply to this item at:

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

___
  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 #2793] Update comments to configure.ac

2011-07-11 Thread Marko Lindqvist

URL:
  http://gna.org/patch/?2793

 Summary: Update comments to configure.ac
 Project: Freeciv
Submitted by: cazfi
Submitted on: Tue 12 Jul 2011 07:32:45 AM EEST
Category: bootstrap
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.4.0

___

Details:

This patch adds some comments to configure.ac to help with cleanup when our
minimum tool requirements are increased.




___

File Attachments:


---
Date: Tue 12 Jul 2011 07:32:45 AM EEST  Name: CacComments.diff  Size: 715B  
By: cazfi

http://gna.org/patch/download.php?file_id=13533

___

Reply to this item at:

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

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


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