<div dir="ltr"><div><div>Thomas: Apologies. This is the right link: <a href="https://github.com/anandkp92/waf/blob/waf/gui/menu.py">https://github.com/anandkp92/waf/blob/waf/gui/menu.py</a><br><br></div>Thanks for the other input. I'll take care of in-code documentation and making the code modular. <br></div><div><br>I'll work on the tabs next. <br><br></div><div>Regards,<br></div><div>Anand<br></div><div><div><div><br></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 16, 2015 at 4:37 AM, Gedare Bloom <span dir="ltr"><<a href="mailto:gedare@gwu.edu" target="_blank">gedare@gwu.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I just had one comment from reading the code, it lacks any code-level<br>
documentation. Granted it is simple now, but as it grows so should the<br>
doc.<br>
<br>
Also, make sure to watch out for when to break out into more<br>
functions/modules. Avoid the pitfall of "one block of code to rule<br>
them all"<br>
<br>
-Gedare<br>
<div><div class="h5"><br>
On Mon, Jun 15, 2015 at 6:49 PM, Amar Takhar <<a href="mailto:amar@rtems.org">amar@rtems.org</a>> wrote:<br>
> On 2015-06-15 23:52 +0530, Anand Krishnan wrote:<br>
>> Hi all,<br>
>><br>
>> Currently I have uploaded the first part of the gui - just plain menus (with no<br>
>> event handlers yet) to my github: <a href="http://github.com/anandkp92/waf/gui/menu.py" rel="noreferrer" target="_blank">github.com/anandkp92/waf/gui/menu.py</a> . The<br>
>> purpose of the GUI is for creating the config.cfg file for "waf config" of the<br>
>> waf version of RTEMS.<br>
><br>
> Looks good so far.<br>
><br>
><br>
>> I think you might need wxpython also to view it right now: http://<br>
>> <a href="http://www.pygopar.com/installing-wxpython-3-on-ubuntu14-04/" rel="noreferrer" target="_blank">www.pygopar.com/installing-wxpython-3-on-ubuntu14-04/</a><br>
><br>
> Everyone here will be able to sort that out, thanks for the help. :)<br>
><br>
><br>
>> The main thing I want to get reviewed on is the code. I am planning to have<br>
>> different classes for menus, tabs, groups etc and a main class that creates<br>
>> multiple instances of each and puts it all together.<br>
><br>
> It seems like a good start.<br>
><br>
><br>
>> Correct me if I am wrong:<br>
>> For the File options-><br>
>> 1) New ->  Opens a new instance with all the configuration options set to<br>
>> default again.<br>
><br>
> Yes.<br>
><br>
><br>
>> 2) Save -> Save the configuration options chosen so far.<br>
><br>
> Yes.<br>
><br>
><br>
>> 3) Open -> Open a previously saved set of configuration options.<br>
><br>
> Yep.<br>
><br>
><br>
>> Once this is cleared, I shall move on to implementing these.<br>
><br>
> OK.<br>
><br>
><br>
>> Please keep in mind this is a very crude version and the first part of many<br>
>> parts.<br>
><br>
> Right.  Lay out the panes and other UI Details before getting on to actual<br>
> options.  I will help you with that.<br>
><br>
> You will also need to decide how each option should be displayed.<br>
><br>
><br>
> Amar.<br>
><br>
</div></div>> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div><br></div>