[PATCH] D61281: [clang-format] Fixed self assignment

This revision was automatically updated to reflect the committed changes. Closed by commit rC360810: [clang-format] Fixed self assignment (authored by xbolva00, committed by ). Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61281/new/ https://reviews.llvm.org/D6128

[PATCH] D61281: [clang-format] Fixed self assignment

MyDeveloperDay accepted this revision. MyDeveloperDay added a comment. This revision is now accepted and ready to land. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61281/new/ https://reviews.llvm.org/D61281 ___ cfe-commits mailing li

[PATCH] D61281: [clang-format] Fixed self assignment

xbolva00 updated this revision to Diff 199494. xbolva00 added a comment. removed line CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61281/new/ https://reviews.llvm.org/D61281 Files: lib/Format/FormatTokenLexer.cpp Index: lib/Format/FormatTokenLexer.cpp =

[PATCH] D61281: [clang-format] Fixed self assignment

MyDeveloperDay requested changes to this revision. MyDeveloperDay added a comment. This revision now requires changes to proceed. please simply remove line 249 Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61281/new/ https://reviews.llvm.org/D61281 ___

[PATCH] D61281: [clang-format] Fixed self assignment

MyDeveloperDay added a comment. 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 mayb

[PATCH] D61281: [clang-format] Fixed self assignment

RKSimon added a reviewer: RKSimon. RKSimon added a comment. 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

[PATCH] D61281: [clang-format] Fixed self assignment

MyDeveloperDay added a comment. Did this case some issue? Does this fix something if so can we add a test, because maybe the line isn't needed Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61281/new/ https://reviews.llvm.org/D61281 ___

[PATCH] D61281: [clang-format] Fixed self assignment

xbolva00 created this revision. xbolva00 added a reviewer: MyDeveloperDay. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rC Clang https://reviews.llvm.org/D61281 Files: lib/Format/FormatTokenLexer.cpp Index: lib/Format/FormatTokenLexer.cpp =