After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 407453 - gtali shows "Roll 2/3" at end of game
gtali shows "Roll 2/3" at end of game
Status: RESOLVED FIXED
Product: gnome-games-superseded
Classification: Deprecated
Component: gtali
2.16.x
Other Linux
: Normal minor
: ---
Assigned To: GNOME Games maintainers
GNOME Games maintainers
Depends on:
Blocks:
 
 
Reported: 2007-02-13 13:01 UTC by Josselin Mouette
Modified: 2007-05-21 18:29 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16


Attachments
Track the roll count to display a meaningful label at game end (2.25 KB, patch)
2007-05-07 10:15 UTC, Jason McFarlane
none Details | Review
Track the roll count to display a meaningful label at game end (2.28 KB, patch)
2007-05-08 09:01 UTC, Jason McFarlane
none Details | Review

Description Josselin Mouette 2007-02-13 13:01:51 UTC
[ forwarded from http://bugs.debian.org/410452 ]

After playing the last move (typically roll 3) and seeing the message
about the winner, the display always shows "Roll 2/3".  Either 1/3 or
3/3 would make more sense.
Comment 1 Jason McFarlane 2007-05-06 09:01:40 UTC
I am currently working on this bug.
Comment 2 Jason McFarlane 2007-05-07 10:15:30 UTC
Created attachment 87689 [details] [review]
Track the roll count to display a meaningful label at game end

If the delay option is turned on, then we see all rolls (including computer rolls) and the final roll label is just set to 3 / 3.
If the delay option is turned off, then we only see the human player rolls so we keep track of the roll count of the last human player to display at the end of the game, instead of showing a roll count modified by a computer player.
Comment 3 Thomas Andersen 2007-05-07 20:47:23 UTC
I've tested the patch and it works as advertised.
There is one small regression though. Playing multiplayer with no computers and delay turned on will also default to 3/3. (not that it makes any sense, but...)

Perhaps change 
    if (DoDelay)
to 
    if (DoDelay && numberOfComputers > 0)
?
Comment 4 Jason McFarlane 2007-05-08 09:01:54 UTC
Created attachment 87787 [details] [review]
Track the roll count to display a meaningful label at game end

Track the roll count to display a meaningful label at game end.
Added additional change to handle case where there are no computer players. 
Thanks Thomas for testing this patch and for the additional suggestion. :)
Comment 5 Andreas Røsdal 2007-05-21 18:29:02 UTC
Thanks for the patch, it has now been committed to SVN:

http://svn.gnome.org/viewcvs/gnome-games?view=revision&revision=6310