Author: krasimir Date: Wed Feb 8 08:45:19 2017 New Revision: 294457 URL: http://llvm.org/viewvc/llvm-project?rev=294457&view=rev Log: [clang-format] Move OriginalPrefix from base to subclass.
Summary: OriginalPrefix is only needed for line comment sections. Moved from the base class to the child class. No functional changes. Reviewers: djasper Reviewed By: djasper Subscribers: cfe-commits, klimek Differential Revision: https://reviews.llvm.org/D29716 Modified: cfe/trunk/lib/Format/BreakableToken.h Modified: cfe/trunk/lib/Format/BreakableToken.h URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Format/BreakableToken.h?rev=294457&r1=294456&r2=294457&view=diff ============================================================================== --- cfe/trunk/lib/Format/BreakableToken.h (original) +++ cfe/trunk/lib/Format/BreakableToken.h Wed Feb 8 08:45:19 2017 @@ -282,10 +282,6 @@ protected: // line. bool FirstInLine; - // In case of line comments, holds the original prefix, including trailing - // whitespace. - SmallVector<StringRef, 16> OriginalPrefix; - // The prefix to use in front a line that has been reflown up. // For example, when reflowing the second line after the first here: // // comment 1 @@ -394,6 +390,14 @@ private: unsigned getContentStartColumn(unsigned LineIndex, unsigned TailOffset) const override; + // OriginalPrefix[i] contains the original prefix of line i, including + // trailing whitespace before the start of the content. The indentation + // preceding the prefix is not included. + // For example, if the line is: + // // content + // then the original prefix is "// ". + SmallVector<StringRef, 16> OriginalPrefix; + // Prefix[i] contains the intended leading "//" with trailing spaces to // account for the indentation of content within the comment at line i after // formatting. It can be different than the original prefix when the original _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
