Re: [PATCH] D22810: scan-build: Add an option to show the description in the list of defect

2016-07-29 Thread Anna Zaks via cfe-commits
zaks.anna accepted this revision. zaks.anna added a comment. This revision is now accepted and ready to land. Ok. LGTM. Thank you! Anna. https://reviews.llvm.org/D22810 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/c

Re: [PATCH] D22810: scan-build: Add an option to show the description in the list of defect

2016-07-27 Thread Sylvestre Ledru via cfe-commits
sylvestre.ledru added a comment. The Firefox report is using a previous version of the patch (which could not be merged upstream: quick and dirty). If you want, I can update my scan-build instance to use the new version (which doesn't have the issues you mentioned) https://reviews.llvm.org/D22

Re: [PATCH] D22810: scan-build: Add an option to show the description in the list of defect

2016-07-27 Thread Anna Zaks via cfe-commits
zaks.anna added a comment. The report you are referencing does not seem to have "Description" as a column name. Also, the titles row is not gray all the way to the right... https://reviews.llvm.org/D22810 ___ cfe-commits mailing list cfe-commits@li

[PATCH] D22810: scan-build: Add an option to show the description in the list of defect

2016-07-26 Thread Sylvestre Ledru via cfe-commits
sylvestre.ledru created this revision. sylvestre.ledru added reviewers: zaks.anna, rizsotto.mailinglist. sylvestre.ledru added a subscriber: cfe-commits. This patch adds an option //--show-description// to add the defect description to the list of defect. This helps to get a better understanding