[GitHub] incubator-quickstep pull request #334: Fix iwyu include path

2018-02-26 Thread jianqiao
GitHub user jianqiao opened a pull request: https://github.com/apache/incubator-quickstep/pull/334 Fix iwyu include path This PR fixes the third-party library include paths for the iwyu (include-what-you-use) tool. You can merge this pull request into a Git repository by running:

[GitHub] incubator-quickstep issue #334: Fix iwyu include path

2018-02-26 Thread hbdeshmukh
Github user hbdeshmukh commented on the issue: https://github.com/apache/incubator-quickstep/pull/334 Hi @jianqiao This looks good to me. Have you been actively using the IWYU library? I stopped using it a while ago because of the false positives. I am not sure if it has

Question about using Intel TBB

2018-02-26 Thread Harshad Deshmukh
Hello, I am planning to incorporate Intel's TBB library (https://www.threadingbuildingblocks.org/) in the Quickstep source code as a third party module. TBB provides fast and parallel data structures such as concurrent hash map. Its licence looks alright to me but I want to run it through

Re: Question about using Intel TBB

2018-02-26 Thread Julian Hyde
Apache license is fine. > On Feb 26, 2018, at 9:11 AM, Harshad Deshmukh wrote: > > Hello, > > > I am planning to incorporate Intel's TBB library > (https://www.threadingbuildingblocks.org/) in the Quickstep source code as a > third party module. TBB provides fast and