06 Jul

18264

0

GSOC Project TSE Week 5

Image: 

Hey there! 0/

Last time you must have seen some snapshots of where the QuickLaunch has reached. Seeing the UI and some functionalities in the snaps must have given you a feeling that it's complete and finished. But, Nah... a lot of work is left, besides debugging so that Dr. Watson would finally be free of his postmortems!! XP

Coming straight to the point, this week the best thing was about 'Code Reviews'. Since this is my first project with ReactOS, I experienced the code reviews for the first time. As you must have guessed, the Crucible platform was used for the same. One of the things which I liked about such code reviews is, everyone has the complete chance to transparently go through the code and comment on whatever part of the code they feel needs correction, according to their own expertise and experience. Thus all doubts get cleared there itself and besides the author gets such a user-friendly UI to handle all the issues and doubts raised in the review. Thanks to Crucible for that and of course ReactOS too. :)

Most of the time of this week was focused on the code reviews but I also continued my work on menus which I had started last time. I implemented the IContextMenu:: QueryContextMenu method to support the working of various options like 'views' and 'show text'. As you can see (see pics below), now the familiar options of the toolbar context menu are working:-

1. Default: week5.1 2. Show Text (disabled): week5.2 3. View - Large Icons: week5.3 4. Both: week5.4

As they say, everyone has a unique pair of eyes and seeing the number of reviews and corrections made really proves this point. And it really helped a lot to improve the quality of code and be in sync with the recommended coding style. You can check the 'CR-122' to get a complete picture of the reviews made. Some of the minor issues which I cleared first was regarding the whitespaces and coding styles. Then I worked on refactoring as suggested. Later I worked on some major code fixes like removing unnecessary Interfaces and be in sync with the 'com_apitests'. One important thing which I learned was about localizing the menus and strings as resources so that it will help in supporting various languages in the future as required.

I thank everyone from ReactOS who helped me by giving a thorough review of my code. See you next time. ;)

#Some Important links to fast track:
  1. List of Milestones and plan
  2. Present history of commits
  3. Code Repository
  4. Code Reviews
This blog post represents the personal opinion of the author and is not representative of the position of the ReactOS Project.

The Blog Posts

Opinions, technical details, side projects or lovely kittens created directly by the ReactOS Devs.

Their opinions are theirs, so...well...any injuries, wounds, or dead-kittens due them, are..well..their own responsibility.

Visit the "Project News" for official statements.