On Mon, Jan 7, 2013 at 6:04 AM, Tim Northover <[email protected]> wrote: > Hi Dmitri, > > Thanks for your comments. I'll fix the issues before sending an updated patch > (I won't send immediately, the patches are quite large and I don't want to > Spam the list).
Perhaps using the Phabricator code review ( http://llvm-reviews.chandlerc.com/ ) would avoid some of the pain of sending large patch attachments to the list repeatedly. (or maybe even a github clone, etc - not sure about that though) > > On Monday 07 Jan 2013 13:56:23 you wrote: >> On Mon, Jan 7, 2013 at 2:57 PM, Tim Northover <[email protected]> wrote: >> +/// UnknownPadding - Return the worst case padding that could result from > >> When adding new code, please follow the new guidelines about doxygen >> documentation: don't duplicate function and class names in the >> comment. Thank you. > > Of course. I thought I'd caught all of these at the end of last year. I may > well have forgotten Clang, I'll check again. > > Regards. > > Tim. > > -- IMPORTANT NOTICE: The contents of this email and any attachments are > confidential and may also be privileged. If you are not the intended > recipient, please notify the sender immediately and do not disclose the > contents to any other person, use it for any purpose, or store or copy the > information in any medium. Thank you. > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
