On Mon, Feb 9, 2009 at 4:22 PM, D. S. <[email protected]> wrote: > > Please, upload clean version of this patch to > http://codereview.appspot.com/ > > Choose "sipXtapi/main branch" and add me (Alexander.Chemeris) > > as a reviewer. > > Thanks for the tip. > I don't submit many patches but I'm learning.
This is new for us too, and we're just starting to use it. http://codereview.appspot.com/ is Google's public install of the Rietveld patch review system (which is open source based derivative of Google's internal Mondrian code review system) I'm in midst of using it for the first such patch review, with Alex right at the moment. It looks very promising for being both a learning/teaching tool for those that are not super-comfortable modifying and providing patches for open source software, and a way to be much more productive and provide better audit trail for code reviews. -- Keith Kyzivat SIPez LLC. SIP VoIP, IM and Presence Consulting http://www.SIPez.com tel: +1 (617) 273-4000
_______________________________________________ sipxtapi-dev mailing list [email protected] List Archive: http://list.sipfoundry.org/archive/sipxtapi-dev/
