[clang-format] Fixed self assignment

[MyDeveloperDay]

Did this cause some issue? Does this fix something if so can we add a test, because maybe the line isn't needed

I would think we'd want to keep this as an identifier. we are just treating arg? the same as we would arg

[RKSimon]

In D61281#1485833, @MyDeveloperDay wrote:

Did this cause some issue? Does this fix something if so can we add a test, because maybe the line isn't needed

I would think we'd want to keep this as an identifier. we are just treating arg? the same as we would arg

@MyDeveloperDay didn't you write this code in D58404?

[MyDeveloperDay]

In D61281#1488022, @RKSimon wrote:

In D61281#1485833, @MyDeveloperDay wrote:

Did this cause some issue? Does this fix something if so can we add a test, because maybe the line isn't needed

I would think we'd want to keep this as an identifier. we are just treating arg? the same as we would arg

@MyDeveloperDay didn't you write this code in D58404?

I did, but I think the correct code is to remove line 249 completely and not to set the token type to the question mark, unless there was an issue which suggested the otherway in which case it should be added as a test, but I think something like PVS-Studio was used to detect this as an issue but then actually the wrong assumption is being made as to what was intended. (which is more dangerous than the self assignment in the first place)

[MyDeveloperDay]

please simply remove line 249

[xbolva00]

removed line

[MyDeveloperDay]

LGTM

lib/Format/FormatTokenLexer.cpp

bool FormatTokenLexer::tryMergeCSharpNullConditionals() {
   auto &Question = *(Tokens.end() - 1);
   if (!Identifier->isOneOf(tok::r_square, tok::identifier) ||
       !Question->is(tok::question))
     return false;
   Identifier->TokenText =
       StringRef(Identifier->TokenText.begin(),
                 Question->TokenText.end() - Identifier->TokenText.begin());
   Identifier->ColumnWidth += Question->ColumnWidth;
-  Identifier->Type = Identifier->Type;
   Tokens.erase(Tokens.end() - 1);
   return true;
 }
 
 bool FormatTokenLexer::tryMergeLessLess() {
   // Merge X,less,less,Y into X,lessless,Y unless X or Y is less.
   if (Tokens.size() < 3)
     return false;