← Back to team overview

gtg team mailing list archive

Re: [Merge] lp:~qcxhome/gtg/bugfix-1218708 into lp:gtg

 


On Fri, Aug 30, 2013 at 4:13 PM, Xuan (Sean) Hu <huxuan8218528@xxxxxxxxx 
<mailto:huxuan8218528@xxxxxxxxx>> wrote:

     > In this case, line 163 in the code is not proper in my opinion
    due to not all potential configuration value has the same format as
    position and size. It is also necessary to consider the generic of
    TaskConfig's interface.

    AFAIK, task configuration only store position and size which are
    both a tuple with two numbers.
    But actually, I agree with you in some way. What do you think, izidor?


FYI, to free consumer from caring about the detail of each configuration 
value, duck typing might be helpful. Within method __getitem__, convert 
each configuration value to proper on in the correct format.

    --
    https://code.launchpad.net/~qcxhome/gtg/bugfix-1218708/+merge/183075
    <https://code.launchpad.net/%7Eqcxhome/gtg/bugfix-1218708/+merge/183075>
    You are the owner of lp:~qcxhome/gtg/bugfix-1218708.




-- 
A Python developer living in Beijing
http://about.me

https://code.launchpad.net/~qcxhome/gtg/bugfix-1218708/+merge/183075
Your team Gtg developers is requested to review the proposed merge of lp:~qcxhome/gtg/bugfix-1218708 into lp:gtg.


References