[patch] If a function decl cannot be merged, mark it as invalid.
klimek at google.com
Tue Jan 20 05:46:45 PST 2015
On Fri Jan 16 2015 at 11:00:06 PM Nico Weber <thakis at chromium.org> wrote:
> On Fri, Jan 16, 2015 at 1:52 PM, Reid Kleckner <rnk at google.com> wrote:
>> On Fri, Jan 16, 2015 at 1:13 PM, Nico Weber <thakis at chromium.org> wrote:
>>> ps: I tried to upload this to phab since this looks like the type of
>>> patch it's good for, but it didn't want to accept richard at metafoo.co.uk
>>> (or richard-llvm at metafoo.co.uk) in the "Reviewers" field; there also
>>> weren't any "Richard Smith"s in the autocomplete field :-/
>> Phab usually wants svn committer ids, like rnk, rsmith, thakis, compnerd,
> Aha, thanks! Phab does seem to know that "rsmith" is also called "Richard
> Smith", at least it shows that in parens in the autocomplete menu. It'd be
> cool if it could autocomplete "rsmith" if I type in "Richard Smith", given
> that it has the data for that. Manuel, could you route that upstream?
When I type "Richard " in the reviewer box I get "rsmith (Richard Smith)"
as one of the proposals; is that not what you're seeing?
> If anyone would rather look at this diff on phab, it's now also on
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits