lp:~pali/llvm/clang-tools-extra-trunk

Created by Pali on 2014-04-12 and last modified on 2017-06-16
Get this branch:
bzr branch lp:~pali/llvm/clang-tools-extra-trunk

Branch merges

Related bugs

Related blueprints

Branch information

Owner:
Pali
Project:
LLVM
Status:
Development

Import details

Import Status: Reviewed

This branch is an import of the Subversion branch from http://llvm.org/svn/llvm-project/clang-tools-extra/trunk.

The next import is scheduled to run in 1 hour.

Last successful import was 3 hours ago.

Import started 4 hours ago on russkaya and finished 3 hours ago taking 50 seconds — see the log
Import started 10 hours ago on russkaya and finished 9 hours ago taking 40 seconds — see the log
Import started 16 hours ago on pear and finished 16 hours ago taking 25 seconds — see the log
Import started 22 hours ago on russkaya and finished 22 hours ago taking 50 seconds — see the log
Import started on 2017-06-22 on russkaya and finished on 2017-06-22 taking 40 seconds — see the log
Import started on 2017-06-22 on russkaya and finished on 2017-06-22 taking 50 seconds — see the log
Import started on 2017-06-21 on pear and finished on 2017-06-21 taking 25 seconds — see the log
Import started on 2017-06-21 on pear and finished on 2017-06-21 taking 25 seconds — see the log
Import started on 2017-06-21 on pear and finished on 2017-06-21 taking 30 seconds — see the log
Import started on 2017-06-21 on pear and finished on 2017-06-21 taking 25 seconds — see the log

Recent revisions

2468. By lebedevri on 2017-06-16

[clang-tidy] readability-function-size: fix nesting level calculation

Summary:
A followup for D32942.

Malcolm Parsons has provided a valid testcase that the initial version of the check complained about nested `if`'s.
As it turns out, the culprit is the **partially** un-intentional `switch` fallthrough.
So rewrite the NestingThreshold logic without ab-using+mis-using that switch with fallthrough, and add testcases with nested `if`' where there should be a warning and shouldn't be a warning. This results in a cleaner, simpler code, too.

I guess, now it would be actually possible to pick some reasonable default for `NestingThreshold` setting.

Fixes PR33454.

Reviewers: malcolm.parsons, alexfh

Reviewed By: malcolm.parsons

Subscribers: sbenza, xazax.hun, cfe-commits, aaron.ballman

Tags: #clang-tools-extra

Differential Revision: https://reviews.llvm.org/D34202

2467. By krasimir on 2017-06-15

[clangd] Add priority to completion item sort text

Summary:
This patch adds the priority of a completion item to the sort text of the
returned LSP result.

Reviewers: ilya-biryukov

Reviewed By: ilya-biryukov

Subscribers: cfe-commits

Tags: #clang-tools-extra

Differential Revision: https://reviews.llvm.org/D34137

2466. By erichkeane on 2017-06-15

Update Append Argument to more efficiently traverse tokens

This function was previously making (correct) assumptions
without complete knowledge of MacroArgs guarantees for
Arguments. After going through Macro Args a bunch, I'd
corrected the getNumArguments (and changed its name),
however didn't realize this was depending on the behavior.

This patch has version that depends on the corrected
getNumMacroArguments's behavior, with the rest checked against
my knowledge of the MacroArgs' token list. Commiting no-wait
since the test is broken.

2465. By erichkeane on 2017-06-14

Update callbacks tracker to match change in 305425

2464. By ibiryukov on 2017-06-14

[clangd] Add a filename parameter to FileSystemProvider.

Reviewers: krasimir

Reviewed By: krasimir

Subscribers: klimek, cfe-commits

Tags: #clang-tools-extra

Differential Revision: https://reviews.llvm.org/D34151

2463. By ibiryukov on 2017-06-13

[clangd] A comment for ClangdServer's constructor. NFC.

2462. By ibiryukov on 2017-06-13

[clangd] Store references instead of unique_ptrs in ClangdServer.

Summary:
ClangdServer owned objects passed to it in constructor for no good reason.
Lots of stuff was moved from the heap to the stack thanks to this change.

Reviewers: krasimir

Reviewed By: krasimir

Subscribers: klimek, cfe-commits

Differential Revision: https://reviews.llvm.org/D34148

2461. By ibiryukov on 2017-06-13

[clangd] Allow to override contents of the file during completion.

Summary:
This is a reapplied r305280 with a fix to the crash found by build bots
(StringRef to an out-of-scope local std::string).

Reviewers: krasimir

Reviewed By: krasimir

Subscribers: klimek, cfe-commits

Differential Revision: https://reviews.llvm.org/D34146

2460. By ibiryukov on 2017-06-13

Revert "[clangd] Allow to override contents of the file during completion."

This caused buildbots failures, reverting until we'll find out what's
wrong.

2459. By ibiryukov on 2017-06-13

[clangd] Allow to override contents of the file during completion.

Reviewers: krasimir

Reviewed By: krasimir

Subscribers: klimek, cfe-commits

Differential Revision: https://reviews.llvm.org/D34107

Branch metadata

Branch format:
Branch format 7
Repository format:
Bazaar repository format 2a (needs bzr 1.16 or later)
This branch contains Public information 
Everyone can see this information.

Subscribers

No subscribers.