On 11/29/2010 7:00 PM, david blanchard wrote:
Hi,
I did a review of the milestones today and here is the pending work
according to me (you can also find it on the wiki) :
1.2 :
- As pending here, I only have the code review, is it ok for you Xav ?
If ok, we can validate the milestone
2.0 :
- As pending here, I only have the code review, is it ok for you Xav
? (see http://farsides.com/forum/index.php?t=msg&th=4789&start=0&
<http://farsides.com/forum/index.php?t=msg&th=4789&start=0&>) If
ok, we can validate the milestone
2.1 :
- I have 4 pending bugs : 676880, 676892, 676908, 682731. And the code
review (Xav ?)
2.2 :
- I have 4 pending bugs : 677066, 612006, 676556, 676547,
- code review - Xav ?
- release note : the release note is not updated with the latest content
Vlad, is this what you also have on your todo list ? What is the
latest ETA to finish this according to you ?
[vlad] For the code review i still have to move some code from the
mission abstract class to the mission engine class (now called mission
action).
this presents a problem which i'll describe here, maybe Xavier can give
his input: so, i have a mission engine which should be resposible fror
manipulating missions. In my mind this will also include the process of
serialization / deserialization of missions and read/write the state to
the db. In order for that to work, the mission must contain information
about the actions that its currently waiting on (the action queue). But
that's the exact code i must take out from the mission class and move it
to the mission engine class. If i do that, then i will have to serialize
the entire mission engine class which doesn't seem like a good decision
from the application design standpoint. So at this point, i do not know
if i still should move the code and make the necessary adjustments or
not? Xavier, what do you think?
On the bugs: from the 2.1 i got some questions for the first 2 bugs
(they are on the bug tracker)
from 2.2 i got questions about 1 bug and i still didn't find a solution
to be able to test the IE6 bug.
Also i have on my list the cleanup that i have to do after all is in
order (tab formatting and other file formatting stuff)
I will be working on the bugs for now and i'll wait for Xavier's answer
on the code review question.
I'm hoping to finish the bugs tomorrow
_______________________________________________
Farsides mailing list - [email protected]
Wiki: http://farsides.com/
List: http://farsides.com/ml/
Forum: http://farsides.com/forum/
Ideas: http://farsides.com/ideas/
Chat: http://farsides.com/chat/
