patchBattle for Wesnoth - Patches: patch #3215, Implementation of duration= in...

Show feedback again

patch #3215: Implementation of duration= in [message]

Submitted by:  Thomas Martinet <hankerspace>
Submitted on:  Wed Mar 28 19:52:17 2012  
Priority: 5 - NormalStatus: In Progress
Privacy: PublicAssigned to: Mark de Wever <mordante>
Open/Closed: Open

Add a New Comment (Rich MarkupRich Markup):

You are not logged in

Please log in, so followups can be emailed to you.


(Jump to the original submission Jump to the original submission)

Wed Apr 18 08:59:25 2012, comment #8:

Updated as auto close timeout final
Using lexical_cast_default and with good indentation.

(file #15596)

Thomas Martinet <hankerspace>
Tue Apr 3 09:59:12 2012, comment #7:

Updated as Auto_Close_Timeout

(file #15506)

Thomas Martinet <hankerspace>
Mon Apr 2 17:09:21 2012, comment #6:

Can you summarize which modifications left ?
Maybe the underscores ?

Thomas Martinet <hankerspace>
Mon Apr 2 16:55:35 2012, comment #5:

Any progress regarding what we discussed on IRC?

Mark de Wever <mordante>
Project MemberIn charge of this item.
Sat Mar 31 18:28:05 2012, comment #4:

Updated with MaxDuration1.patch

(file #15487)

Thomas Martinet <hankerspace>
Fri Mar 30 21:00:19 2012, comment #3:

I've use copy and paste to respect the indentation and rename my variable. I note your advice. Thanks.

(file #15470)

Thomas Martinet <hankerspace>
Fri Mar 30 18:35:56 2012, comment #2:

Some notes on coding conventions:
We like to have code written in a more-or-less same style.

non-public class member variables (and only them) should end in underscore (like 'max_duration_' when it is a class variables, and just 'max_duration' when it is a local variable or a method parameter)

So, use max_duration_ only when it is a class variable, and never use _max_duration

Also, watch the spacing - use only tabs, not spaces, to indent C++ code in wesnoth.

Iurii Chernyi <crab>
Project Member
Thu Mar 29 10:37:55 2012, comment #1:

Updated with shadowm and anonymissimus advises :
-use lexical_cast instead atoi()
-use the "max_" prefix for variables

(file #15448)

Thomas Martinet <hankerspace>
Wed Mar 28 19:52:17 2012, original submission:

An implementation of duration, in frames. When a duration=X is specified in [message], the message will automatically dissapear after X frames. If the player click or press spacebar, the message can be skipped earlier.

Was requested here :

Thomas Martinet <hankerspace>


(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):

Attached Files
file #15487:  MaxDuration1.patch added by hankerspace (3kB - application/octet-stream - Last update)
file #15473:  MaxDuration.patch added by hankerspace (4kB - application/octet-stream)


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -unavailable- added by mordante (Posted a comment)
  • -unavailable- added by crab (Posted a comment)
  • -unavailable- added by hankerspace (Submitted the item)

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.


    Error: not logged in



    Follow 10 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed Apr 18 08:59:25 2012hankerspaceAttached File-=>Added Auto close timeout final.patch, #15596
    Tue Apr 3 09:59:12 2012hankerspaceAttached File-=>Added Auto_Close_Timeout.patch, #15506
    Mon Apr 2 16:55:35 2012mordanteStatusNone=>In Progress
      Assigned toNone=>mordante
    Sat Mar 31 18:28:05 2012hankerspaceAttached File-=>Added MaxDuration1.patch, #15487
    Fri Mar 30 22:13:02 2012hankerspaceAttached File-=>Added MaxDuration.patch, #15473
    Fri Mar 30 21:13:06 2012hankerspaceAttached File-=>Added ImplementationDurationFinal (1).patch, #15471
    Fri Mar 30 21:00:19 2012hankerspaceAttached File-=>Added ImplementationDurationFinal.patch, #15470
    Thu Mar 29 10:37:55 2012hankerspaceAttached File-=>Added ImplementationDurationUPDATED.patch, #15448
    Wed Mar 28 19:52:17 2012hankerspaceAttached File-=>Added ImplementationDuration.patch, #15444
    Show feedback again

    Back to the top

    Powered by Savane 3.1-cleanup