On Wed, Mar 25, 2015 at 8:34 AM, Olle E. Johansson <[email protected]> wrote: > Friends, > Going through some Asterisk 11 code for my RTCPFB work. There are a lot of > new code in the RTP module - almost zero comments. Those that are there > are generally not doxygen formatted. > > Can we please try to add more comments as you add new code? Please. Names > doesn't explain your logic and you will soon grow old and will not > remember how stuff worked or why you added that super function? >
Which functions are you referring to specifically? If they're missing documentation, we can try and get that fixed up. -- Matthew Jordan Digium, Inc. | Director of Technology 445 Jan Davis Drive NW - Huntsville, AL 35806 - USA Check us out at: http://digium.com & http://asterisk.org -- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
