Update of bug #15708 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Follow-up Comment #1:
Fix committed to
Update of bug #15705 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
Update of bug #15738 (project freeciv):
Status:None = Ready For Test
Assigned to:None = englabenny
___
Reply to this item at:
Update of bug #15738 (project freeciv):
Planned Release: = 2.2.1, 2.3.0
___
Reply to this item at:
http://gna.org/bugs/?15738
___
Meddelandet
Update of bug #15706 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
Update of bug #15707 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
Update of bug #15644 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
Update of bug #15674 (project freeciv):
Status:None = Fixed
Open/Closed:Open = Closed
Planned Release: = 2.2.1, 2.3.0
Update of patch #1263 (project freeciv):
Summary: possibility to loose a tech if it is transfered =
possibility to lose a tech if it is transfered
___
Follow-up Comment #3:
This spelling error is very common everywhere
Follow-up Comment #3, patch #1494 (project freeciv):
Here is a port of Maho's newcomers code to C. In C, we can finally read and
understand it :-) It is a diff against trunk rev 17191
It seems to work well now. The critical parts come when the player list is
already full and dead players have
Update of bug #15663 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
Update of bug #15729 (project freeciv):
Status:None = Fixed
Open/Closed:Open = Closed
___
Follow-up Comment #1:
Close, since this
Update of bug #15738 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
URL:
http://gna.org/patch/?1569
Summary: Fix misspellings of lose or loser in trunk
Project: Freeciv
Submitted by: englabenny
Submitted on: tisdag 2010-03-30 den 20:45
Category: general
Priority: 5 -
Follow-up Comment #5, patch #1263 (project freeciv):
See gna patch #1569 for a patch to fix the other loose/looser in trunk. Feel
free to commit that patch if you get around to it before me.
___
Reply to this item at:
Follow-up Comment #1, patch #1569 (project freeciv):
Question: One misspelling occurs in a user-visible string. Should it be fixed
in 2.2?
___
Reply to this item at:
http://gna.org/patch/?1569
Update of bug #15738 (project freeciv):
Status: Fixed = Ready For Test
Open/Closed: Closed = Open
___
Follow-up Comment #6:
I see what you
Follow-up Comment #7, bug #15738 (project freeciv):
Or, we do like this, to simply keep the same tile in center while resizing
the mapview. In this patch it is done very formally with
map_keep_center_begin()/_end(), but maybe it should be the default behavior?
This way we reuse the already
Follow-up Comment #5, patch #1494 (project freeciv):
The C version can lets us see what we move to lua easily and what not. But my
previous message already has a proposed plan of attack which mentions which
parts could be implemented in lua and which not.
If maho himself has any input it'd
URL:
http://gna.org/bugs/?15756
Summary: Show if city is occupied or not in Middle-click
popup
Project: Freeciv
Submitted by: englabenny
Submitted on: tisdag 2010-03-30 den 23:14
Category: client-gtk-2.0
Follow-up Comment #6, patch #1494 (project freeciv):
Maybe my original message was not as clear on C vs. lua as I thought, I'm
sorry.
Rough idea might be:
1. /reap server command -- implement in C
2. server command or server function to create a new player during the game,
without giving the
Update of bug #15756 (project freeciv):
Category: client-gtk-2.0 = client
___
Follow-up Comment #2:
The capital city info should already be there, it should list Palace as one
of the wonders in
Follow-up Comment #11, bug #15738 (project freeciv):
Here is a patch to change the mapview's behavior. Proposed for 2.2 and
trunk.
We simply keep the center tile invariant when the mapview resizes. So tile
displayed in the center of mapview before resize == tile in center after.
Notice it does
Follow-up Comment #12, bug #15738 (project freeciv):
Maybe we should change map_canvas_resized instead to keep the center
invariant (instead of the origin)? That would fix recentering issues on for
example tileset change(?).
___
Reply to
Good, I wrote a brief explanation that you can include to explain the
functions (see previous posting). Please include it if you find any
value in it.
ulrik
Den 1 apr 2010 kl. 19.50 skrev pepeto NO-REPLY.INVALID-
addr...@gna.org:
Update of bug #15377 (project freeciv):
Follow-up Comment #24, bug #15377 (project freeciv):
Here it is.
These functions are wrappers for the libc character class functions, without
any locale-dependent behavior.
The character functions work as documented for ASCII. Bytes outside of the
ASCII set will not be reported to belong to
Follow-up Comment #3, patch #1582 (project freeciv):
I forgot to log in, the last comment was from me.
___
Reply to this item at:
http://gna.org/patch/?1582
___
Meddelandet skickades
Follow-up Comment #9, patch #1236 (project freeciv):
The experimental ruleset also serves as documentation or example of ruleset
possibilities, something that might be very helpful for people creating
custom rulesets.
___
Reply to this
Follow-up Comment #20, patch #1330 (project freeciv):
Just my opinion but, the city dialog should not be as small as possible. If
everything is just available via small clickable icons, it's hard to use. The
gtk client uses a lot of space for common tasks (changing production from
first city
Follow-up Comment #16, bug #15738 (project freeciv):
I see that the map is drawn with canvas_put_rectangle in map_canvas_resized.
Does that mean that regardless if moving the origin or not, we still redraw
the whole canvas without moving/copying pixels?
Follow-up Comment #3, patch #1574 (project freeciv):
Nice, the code looks pretty simple actually.
I find the same map inconsistency when reviving a player (creating a new
player in a dead player's slot). Looking at the map editor, it seems the
server thinks the player can see the whole map,
URL:
http://gna.org/patch/?1599
Summary: Scripting: Limit execution time
Project: Freeciv
Submitted by: englabenny
Submitted on: torsdag 2010-04-08 den 00:53
Category: general
Priority: 5 - Normal
Follow-up Comment #1, patch #1599 (project freeciv):
I forgot this remark: This is not a security feature. It is reportedly
possible to construct infinite loops using C library calls, and those won't
be caugth (the instructions counter does not advance).
Follow-up Comment #1, patch #1561 (project freeciv):
Version for 2.2 branch. Without any dependencies.
(file #8864)
___
Additional Item Attachment:
File name: 2010040801-Make-API-objects-fields-immutable.patch Size:1 KB
Update of patch #1561 (project freeciv):
Planned Release: 2.3 = 2.2.1, 2.3.0
___
Reply to this item at:
http://gna.org/patch/?1561
___
Meddelandet
URL:
http://gna.org/patch/?1601
Summary: [Metaticket] Scripting/API extension towards
iteration and querying
Project: Freeciv
Submitted by: englabenny
Submitted on: fredag 2010-04-09 den 09:40
Category: None
URL:
http://gna.org/patch/?1602
Summary: API: Introduce players_iterate() and
whole_map_iterate()
Project: Freeciv
Submitted by: englabenny
Submitted on: fredag 2010-04-09 den 09:41
Category: general
URL:
http://gna.org/patch/?1605
Summary: API: Add Tile:city() and Tile:num_units() accessors
Project: Freeciv
Submitted by: englabenny
Submitted on: fredag 2010-04-09 den 09:43
Category: None
Priority: 5
URL:
http://gna.org/patch/?1606
Summary: API: Add Tile,Player:units_iterate(), and
Player:cities_iterate()
Project: Freeciv
Submitted by: englabenny
Submitted on: fredag 2010-04-09 den 09:46
Category: None
Update of patch #1601 (project freeciv):
Depends on: = patch #1602
___
Reply to this item at:
http://gna.org/patch/?1601
___
Meddelandet
Update of bug #13709 (project freeciv):
Priority: 1 - Later = 5 - Normal
Status: In Progress = Ready For Test
___
Follow-up Comment #1:
Here are patches 1
Follow-up Comment #3, patch #1562 (project freeciv):
Very small patch update for tolua-5.1 (tolua-5.1 can't handle embedded code
block ending in comment)
(file #8879)
___
Additional Item Attachment:
File name:
Update of patch #1604 (project freeciv):
Status: Need Info = Ready For Test
___
Reply to this item at:
http://gna.org/patch/?1604
___
Meddelandet
Update of patch #1606 (project freeciv):
Status: Need Info = Ready For Test
___
Reply to this item at:
http://gna.org/patch/?1606
___
Meddelandet
Update of patch #1602 (project freeciv):
Status:None = Ready For Test
___
Reply to this item at:
http://gna.org/patch/?1602
___
Meddelandet
Follow-up Comment #4, patch #1562 (project freeciv):
Sorry for the spam. Yet another update for Lua-5.1, '.string' metamethod does
not exist, we use '__tostring' instead.
(file #8880)
___
Additional Item Attachment:
File name:
Update of patch #1561 (project freeciv):
Status: Ready For Test = Postponed
___
Reply to this item at:
http://gna.org/patch/?1561
___
Meddelandet
Follow-up Comment #7, patch #1574 (project freeciv):
Here is how to add a function to the API. It's called 'create_player' in the
API (rhymes well with create_unit, create_base, etc).
All is needs is that the wrapper function api_actions_create_player in
server/scripting/api_actions.h can find
Follow-up Comment #4, bug #13709 (project freeciv):
Thanks for your thoughts. I've been missing more discussion about script/api
changes.
Given how little that happened with scripting before, and how much that is
possible now given the momentum we have (right now), I think we should just
go
Update of bug #15738 (project freeciv):
Status: Ready For Test = Need Info
Assigned to: englabenny = None
___
Follow-up Comment #17:
I don't think I
URL:
http://gna.org/patch/?1611
Summary: Scripting: Hide Lua module 'debug'
Project: Freeciv
Submitted by: englabenny
Submitted on: lördag 2010-04-10 den 22:47
Category: None
Priority: 5 - Normal
Update of bug #15725 (project freeciv):
Depends on: = patch #1611
___
Reply to this item at:
http://gna.org/bugs/?15725
___
Meddelandet
Update of patch #1599 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #1562 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #1602 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
URL:
http://gna.org/bugs/?15841
Summary: API: Do not use tolua_outside
Project: Freeciv
Submitted by: englabenny
Submitted on: söndag 2010-04-11 den 01:08
Category: None
Severity: 3 - Normal
Update of bug #13709 (project freeciv):
Assigned to: cazfi = englabenny
Depends on: = bugs #15841
___
Follow-up Comment #5:
I hope you don't
Additional Item Attachment, patch #1603 (project freeciv):
File name: 2010041102-API-Add-Player-civilization_score-and-Player-g.patch
Size:3 KB
___
Reply to this item at:
http://gna.org/patch/?1603
Additional Item Attachment, patch #1604 (project freeciv):
File name: 2010041103-API-Add-Tile-square_iterate-.-Tile-circle_iter.patch
Size:6 KB
___
Reply to this item at:
http://gna.org/patch/?1604
Additional Item Attachment, patch #1605 (project freeciv):
File name: 2010041104-API-Add-Tile-city-and-Tile-num_units-accessors.patch
Size:3 KB
___
Reply to this item at:
http://gna.org/patch/?1605
Additional Item Attachment, patch #1606 (project freeciv):
File name: 2010041105-API-Add-Tile-Player-units_iterate-and-Player-c.patch
Size:9 KB
___
Reply to this item at:
http://gna.org/patch/?1606
Follow-up Comment #7, bug #13709 (project freeciv):
That's a good idea. I think we can be without the first patch for
tolua_outside, see bug #15841.
The second patch for tolua_notnil is still interesting since there is a lot
less duplicate code/duplicate checks needed with it. But we don't need
Follow-up Comment #1, patch #1614 (project freeciv):
I guess if caller is handled the same way as for other commands so that
access controls still apply, it will be safe. What happens if caller is
NULL?
*This particular* instance of lua scripting in freeciv is about the game; not
the server,
Follow-up Comment #3, bug #15844 (project freeciv):
Thanks. I saw consistent crashes every time a city was destroyed (trunk)
(either real city destruction in the game or when the client quits). I
didn't have time yet to make a clean checkout to testreport it yet. This
seems to be it, the patch
URL:
http://gna.org/patch/?1617
Summary: Release script objects for removed units and cities
Project: Freeciv
Submitted by: englabenny
Submitted on: söndag 2010-04-11 den 22:10
Category: general
Priority:
Update of patch #1617 (project freeciv):
Depends on: = bugs #13709
___
Reply to this item at:
http://gna.org/patch/?1617
___
Meddelandet
Update of bug #15725 (project freeciv):
Depends on: = patch #1617
___
Reply to this item at:
http://gna.org/bugs/?15725
___
Meddelandet
Follow-up Comment #9, patch #1574 (project freeciv):
I wonder if it should be possible to choose the nation for the player to be
created. It would make sense for scenario scripts. nil/NULL would be random
nation (that part works very well by the way, seems to pick nations of the
same category as
URL:
http://gna.org/patch/?1618
Summary: Scripting/API: Use specific argument error
messages
Project: Freeciv
Submitted by: englabenny
Submitted on: söndag 2010-04-11 den 23:55
Category: None
Priority: 5
Update of patch #1618 (project freeciv):
Depends on: = patch #1601
___
Reply to this item at:
http://gna.org/patch/?1618
___
Meddelandet
Follow-up Comment #1, bug #15841 (project freeciv):
Updated patch, this way it's even better/easier. Will commit soon.
(file #8928)
___
Additional Item Attachment:
File name: 2010041201-API-Do-not-use-tolua_outside.patch Size:6 KB
Follow-up Comment #2, patch #1617 (project freeciv):
Ah ok. I'll look at player in a followup patch. It seems players are never
released, but they are put into an unusable state after being removed so it's
similar.
Attached patch has minor update for change in tolua_outside patch.
(file #8929)
URL:
http://gna.org/bugs/?15857
Summary: Fix buggy find_transport_from_tile
Project: Freeciv
Submitted by: englabenny
Submitted on: måndag 2010-04-12 den 14:05
Category: general
Severity: 3 - Normal
Update of bug #15841 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
Update of bug #13709 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Follow-up Comment #10:
Switched to
Update of bug #13709 (project freeciv):
Status: Fixed = In Progress
Planned Release: 2.3.0 = 2.2.1, 2.3.0
___
Follow-up Comment #14:
The trunk error
URL:
http://gna.org/bugs/?15860
Summary: Fix compiler warning in tolua-5.1
Project: Freeciv
Submitted by: englabenny
Submitted on: måndag 2010-04-12 den 20:48
Category: general
Severity: 3 - Normal
Update of bug #13709 (project freeciv):
Status: In Progress = Fixed
___
Follow-up Comment #17:
The compilation error is tracked in a separate bug #15860
Update of patch #1611 (project freeciv):
Planned Release: 2.3.0 = 2.2.1, 2.3.0
___
Follow-up Comment #1:
This should go into both 2.2 and trunk, since the change is easily available
to make in 2.2 as
Update of patch #1605 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #1606 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Follow-up Comment #4, patch #1617 (project freeciv):
Updating the patch. This makes the mechanism much better.
New summary:
We change the type of the userdata to a marker class called
'Nonexistent' [when a freeciv object such as city or unit is free'd].
Nonexistent has one single method
Update of patch #1617 (project freeciv):
Planned Release: 2.3.0 = 2.2.1, 2.3.0
___
Follow-up Comment #5:
Patch for 2.2 branch attached, adding same object removal and new API for
:exists() methods.
Update of patch #1618 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Follow-up Comment #20, bug #15738 (project freeciv):
The latest pepeto patch works well for me.
___
Reply to this item at:
http://gna.org/bugs/?15738
___
Meddelandet skickades via/av Gna!
Update of bug #15860 (project freeciv):
Assigned to:None = englabenny
___
Reply to this item at:
http://gna.org/bugs/?15860
___
Meddelandet
Follow-up Comment #1, bug #15725 (project freeciv):
Here is a plan:
No unsafe functions (should be done)
No NULL pointers and no wild pointers (in progress):
1. We check arguments everywhere. Script functions should try to assure
success or raise a script error.
2. Debug module hidden to
Follow-up Comment #2, bug #15725 (project freeciv):
To clarify the last comment:
We hide '.get', but the tolua runtime will use it to look up object fields.
This will be type safe, unlike direct access to the .get table.
We remove '.set', so the tolua runtime will not allow writing back object
URL:
http://gna.org/bugs/?15876
Summary: API: Lua part of runtime to be shipped in Freeciv
data directory
Project: Freeciv
Submitted by: englabenny
Submitted on: onsdag 2010-04-14 den 13:23
Category: general
Follow-up Comment #6, patch #1617 (project freeciv):
Yet another version. To be consistent across save/load we have to restore
even 'Nonexistent' values.
Also removing the script_is_exported_object_valid function. It is mostly for
debugging.
Patches for trunk and 2.2
(file #8956, file #8957)
Update of patch #1611 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of bug #15860 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
Update of bug #15857 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Follow-up Comment #3:
This fixes some
Follow-up Comment #2, patch #1611 (project freeciv):
Committed to SVN revs r17341 (trunk) and r17342 (S2_2)
___
Reply to this item at:
http://gna.org/patch/?1611
___
Meddelandet skickades
URL:
http://gna.org/bugs/?15878
Summary: freeciv-server.desktop freeciv.desktop
Project: Freeciv
Submitted by: englabenny
Submitted on: onsdag 2010-04-14 den 16:07
Category: None
Severity: 3 - Normal
Update of bug #15586 (project freeciv):
Status: Fixed = In Progress
Open/Closed: Closed = Open
___
Follow-up Comment #5:
A month wiser, I
Update of patch #1601 (project freeciv):
Summary: [Metaticket] Scripting/API extension towards
iteration and querying = [Metaticket] Scripting/API extension towards
iteration and querying for newcomers
___
Reply to this
URL:
http://gna.org/patch/?1626
Summary: API: Player:knows_tech(..) and
Unit_Type:can_exists_at_tile(..)
Project: Freeciv
Submitted by: englabenny
Submitted on: onsdag 2010-04-14 den 21:00
Category: None
Update of patch #1601 (project freeciv):
Depends on: = patch #1626
___
Reply to this item at:
http://gna.org/patch/?1601
___
Meddelandet
Follow-up Comment #14, patch #1574 (project freeciv):
create_player returns NULL, because the player-creating commands (newcomer
and pregame) only take struct player *, not struct player ** arguments that
are needed to be able to write the pointer back to the caller.
201 - 300 of 350 matches
Mail list logo