[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-17 Thread Daniel Krupp via Phabricator via cfe-commits
dkrupp added inline comments. Comment at: www/analyzer/open_projects.html:198 + or using a dataflow framework. + (Difficulty: Hard) + Probably it is worth mentioning here, that there is a macro language already for describing summaries of standard

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-16 Thread George Karpenkov via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC344663: [analyzer] [www] Updated a list of open projects (authored by george.karpenkov, committed by ). Changed prior to commit: https://reviews.llvm.org/D53024?vs=169920&id=169933#toc Repository: rC

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-16 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ accepted this revision. NoQ added a comment. Ok, let's see if this actually works :) https://reviews.llvm.org/D53024 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-16 Thread George Karpenkov via Phabricator via cfe-commits
george.karpenkov updated this revision to Diff 169920. https://reviews.llvm.org/D53024 Files: clang/www/analyzer/open_projects.html Index: clang/www/analyzer/open_projects.html === --- clang/www/analyzer/open_projects.html +++ cla

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-16 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added inline comments. Comment at: clang/www/analyzer/open_projects.html:27-32 +New checkers which were contributed to the analyzer, +but have not passed a rigorous evaluation process, +are committed as "alpha checkers" (from "alpha version"), +and are not ena

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-16 Thread George Karpenkov via Phabricator via cfe-commits
george.karpenkov added inline comments. Herald added a subscriber: dkrupp. Comment at: clang/www/analyzer/open_projects.html:27-32 +New checkers which were contributed to the analyzer, +but have not passed a rigorous evaluation process, +are committed as "alpha checke

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-16 Thread Umann Kristóf via Phabricator via cfe-commits
Szelethus added a comment. Why d Comment at: clang/www/analyzer/open_projects.html:153 + problem still remains open. + + (Difficulty: Hard) Did you mean to have this newline here? Difficulty seems to have a weird placement when viewed in a browser.

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-15 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. The current text looks great. https://reviews.llvm.org/D53024 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-15 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added inline comments. Comment at: clang/www/analyzer/open_projects.html:27-32 +New checkers which were contributed to the analyzer, +but have not passed a rigorous evaluation process, +are committed as "alpha checkers" (from "alpha version"), +and are not ena

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-15 Thread Umann Kristóf via Phabricator via cfe-commits
Szelethus added a comment. I dislike web development, but that would indeed be invaluable. I'll take a look. https://reviews.llvm.org/D53024 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-15 Thread George Karpenkov via Phabricator via cfe-commits
george.karpenkov added a comment. @Szelethus thanks! BTW if you really want to invest into maintaining the website, I think it's totally worth it to change all contents to markdown, and then have a script to generate HTML from that. Committers would be expected to manually run that script. This w

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-15 Thread Umann Kristóf via Phabricator via cfe-commits
Szelethus accepted this revision. Szelethus added a comment. This revision is now accepted and ready to land. Thanks, this looks great! https://reviews.llvm.org/D53024 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-15 Thread George Karpenkov via Phabricator via cfe-commits
george.karpenkov updated this revision to Diff 169758. https://reviews.llvm.org/D53024 Files: clang/www/analyzer/open_projects.html Index: clang/www/analyzer/open_projects.html === --- clang/www/analyzer/open_projects.html +++ cla

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-15 Thread George Karpenkov via Phabricator via cfe-commits
george.karpenkov added a comment. I have tried to clean up the list. https://reviews.llvm.org/D53024 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-12 Thread Umann Kristóf via Phabricator via cfe-commits
Szelethus added a comment. In https://reviews.llvm.org/D53024#1262976, @george.karpenkov wrote: > @Szelethus I take it this is mostly formed from @NoQ email? Language could > use polishing in quite a few places, could I just commandeer this revision > and try to fix it? Yes it is. Though the

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-11 Thread George Karpenkov via Phabricator via cfe-commits
george.karpenkov added a comment. @Szelethus I take it this is mostly formed from @NoQ email? Language could use polishing in quite a few places, could I just commandeer this revision and try to fix it? Comment at: www/analyzer/open_projects.html:29 +Implement a dataflow

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-11 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. > In addition 2018 Bay Area LLVM Developers' Meetings may bring some new open > projects :) Actually, let's commit this before the conference, even if it's not perfect, so that people who suddenly get inspired to work on Static Analyzer already had an updated list :) ==

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-11 Thread Umann Kristóf via Phabricator via cfe-commits
Szelethus updated this revision to Diff 169193. Szelethus added a comment. - Fixed typos - Fixed stupid characters (Thank you so much @MTC for going the extra mile and opening this patch in a browser!) - Removed outdated entries as mentioned by @MTC https://reviews.llvm.org/D53024 Files: www

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-10 Thread Henry Wong via Phabricator via cfe-commits
MTC added subscribers: teemperor, baloghadamsoftware, blitz.opensource. MTC added a comment. Herald added a subscriber: donat.nagy. In https://reviews.llvm.org/D53024#1258976, @Szelethus wrote: > Also, a lot of items on this list is outdated, but I joined the project > relatively recently, so I'

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-09 Thread Umann Kristóf via Phabricator via cfe-commits
Szelethus added a comment. Thanks! I admit that the difficulty was mostly chosen at random, so that could be brought closer to the actual difficulty of the project. Comment at: www/analyzer/open_projects.html:86-87 +the opposite direction - integers to pointers - are comp

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-09 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. Whoa thanks! Will have a closer look again. Comment at: www/analyzer/open_projects.html:33 + +Handle aggregate construction. +Aggregates are object that can be brace-initialized without calling a I'll try to list other constructor k

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-09 Thread Umann Kristóf via Phabricator via cfe-commits
Szelethus added a comment. Mind you, there are some ideas I didn't add to the list -- I just don't know how to put them to words nicely, but I'm on it. Also, a lot of these is outdated, but I joined the project relatively recently, so I'm not sure what's the state on all of them. Repository:

[PATCH] D53024: [analyzer][www] Add more open projects

2018-10-09 Thread Umann Kristóf via Phabricator via cfe-commits
Szelethus created this revision. Szelethus added reviewers: NoQ, george.karpenkov, xazax.hun, MTC. Herald added subscribers: cfe-commits, jfb, mikhail.ramalho, a.sidorin, rnkovacs, szepet, whisperity. Added some extra tasks to the open projects. These are the ideas of @NoQ and @george.karpenkov,