Page MenuHomePhabricator

dougpuob (Douglas Chen)
User

Projects

User does not belong to any projects.

User Details

User Since
Dec 7 2018, 10:34 PM (132 w, 5 d)

Recent Activity

Thu, Jun 10

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

@njames93, Ping

Thu, Jun 10, 2:01 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

May 16 2021

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Moved all static functions into IdentifierNamingCheck class, and called the configurationDiag() directly in IdentifierNamingCheck::getFileStyleFromOptions() method.
May 16 2021, 1:50 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

May 13 2021

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Hi @njames93:
Could you do me a favor? Because it is my first patch, something I'm not sure. I'm confused about can I land this patch now? I read the "LLVM Code-Review Policy and Practices" document, it said patches can be landed if received a LGTM, but seems you are still reviewing.

If you have made significant changes (excluding what a reviewer asks when giving an LGTM) Its best to get those further changes also reviewed.

May 13 2021, 7:11 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

May 11 2021

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Hi @njames93:
Could you do me a favor? Because it is my first patch, something I'm not sure. I'm confused about can I land this patch now? I read the "LLVM Code-Review Policy and Practices" document, it said patches can be landed if received a LGTM, but seems you are still reviewing.

May 11 2021, 9:21 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

May 10 2021

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Removed diagnoseInvalidConfigOption() function.
  • Created a HungarianNotation class and moved related functions into there.
May 10 2021, 7:43 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

May 8 2021

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

I tried to create a class, HungarianNotation, and put all related functions into the class. Then I can call configurationDiag() in HungarianNotation::checkOptionValid() function.
How about this way?

May 8 2021, 9:05 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

May 2 2021

dougpuob added inline comments to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
May 2 2021, 7:37 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

That's ok. You have helped me a lots. Thank you.

May 2 2021, 12:02 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

May 1 2021

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Against trunk.
  • Refined isHungarianNotationOptionEnabled() function.
  • Changed old way that showing invalid config option in .clang-tidy config file. Now added a new function ClangTidyCheck::OptionsView::diagnoseInvalidConfigOption() instead.
  • Recovered that document content missing in .rst.
May 1 2021, 12:14 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Jan 13 2021

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Ping @njames93

Jan 13 2021, 7:18 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Dec 30 2020

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Addressed comments by @njames93. Including adding warning message for unsupported options in config file, refining code in getFileStyleFromOptions(), and for consistent reason to use llvm::yaml::parseBool() function instead of checking On/Off string.

Dec 30 2020, 1:22 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Dec 29 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Removed unrelated comments.
Dec 29 2020, 11:01 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Against git master.
  • Refined isHungarianNotationOptionEnabled() function.
  • Classify document and test cases.
  • Fixed assertions by clang-format.
  • Instead check On/Off string with llvm::yaml::parseBool() function for consistent.
  • Improved code.
  • Add new check for unsupported option.
Dec 29 2020, 7:50 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Dec 22 2020

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

One last point, is there a way to validate that these options are only set where it makes sense. If someone tries to use say MacroDefinitionHungarianPrefix That could be warning worthy?

Dec 22 2020, 4:40 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Dec 11 2020

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Hi @njames93, thank you for your review suggestions, I have improved them and against my change to the main branch.

Dec 11 2020, 10:25 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Fixed assertions by clang-format.
Dec 11 2020, 9:59 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Against git master.
  • Refined isHungarianNotationOptionEnabled() function.
  • Classify document and test cases.
Dec 11 2020, 6:23 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Dec 6 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

arc diff with the correct base.

Dec 6 2020, 7:01 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Improved code review suggestions from @njames93. Including move the IdentifierNamingCheck::HNOption variable to IdentifierNamingCheck::FileStyle::HNOption, use try_emplace() instead of insert() and lookup().
Dec 6 2020, 6:48 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Dec 1 2020

dougpuob requested review of D92410: Summary: Fixed a broken link to the 'Widening integer arithmetic' paper in document..
Dec 1 2020, 10:41 AM · Unknown Object (Project)

Nov 24 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Improved code review suggestions from Aaron.Ballman and Eugene.Zelenko. Including return default initializer instead of empty string, use isa<>() instead of if statement with multiple conditions, use StringRef array instead of list<std::string>.
Nov 24 2020, 9:20 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Hi @aaron.ballman and @Eugene.Zelenko, thank you for your suggestions. I will improve them and upload my diff later.

Nov 24 2020, 7:56 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Nov 22 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Improved code review suggestions from Aaron Ballman(aaron.ballman). Including document, single-line if statements, no reference with StringRef, "char"-->"char[]" for HNOption.CString, use llvm::transform insteads of for-loop to uppercase string, don't check CXXMethod in getDeclTypeName(), remove const_cast<char *> in getDeclTypeName(), use if-else insteads of switch-case(simplify src).
Nov 22 2020, 5:37 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob removed a project from D86671: [clang-tidy] Add new case type to check variables with Hungarian notation: Restricted Project.
Nov 22 2020, 5:00 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Hi @aaron.ballman, thank you for your feedback. I will update my change later. Unrelated change were mixed with other commits when I against git master. I did it again then the problem was gone. I found the command, arc diff master --preview, knowing exactly changes before uploading diff by arc.

Nov 22 2020, 3:43 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Nov 21 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Against git master.

Nov 21 2020, 11:57 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Nov 19 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Against git master.

Nov 19 2020, 7:32 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Nov 18 2020

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

ping?

Nov 18 2020, 6:04 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Nov 12 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Shortened namespace with type alias.
  • Removed namespace for StringRef.
Nov 12 2020, 10:10 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Nov 4 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Fixed warnings of Clang-Tidy.
Nov 4 2020, 8:38 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  1. Fixed build failure on BuildBot. (Touch empty data from Options.get())
  2. Changed llvm::StringMap with std::pair in getHungarianNotationDefaultConfig().
  3. Moved HNOpts and HNDerviedTypes declarations out of getHungarianNotationDefaultConfig().
  4. Reused SmallString<128> for string concatenation.
  5. Removed redundant m(HungarianNotation) in #define NAMING_KEYS(m).
Nov 4 2020, 7:37 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added inline comments to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
Nov 4 2020, 7:34 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Nov 1 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  1. Applied all patches with the trunk(1800b44651c19b11e7680f908344d5751e8d2246).
  2. Moved HNOption pointer out of NamingStyle data structure.
  3. Removed parseHungarianPrefix() function, use Options.get() instead.
Nov 1 2020, 5:00 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added inline comments to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
Nov 1 2020, 4:58 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Oct 31 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Removed clearAll() function and made the HPOption variable passing by value(origin is std::move()).
  • Moved static HNOption variable from static getNamingStyles() function to IdentifierNamingCheck class a private data member.
  • Replaced auto keyword with explicit std::string.
  • Elided braces around single-line.
  • Used CXXRecordDecl::isAbstract() directly.
  • changed const auto to const auto * for pointer objects.
  • Enhanced test case for enum with two EnumConstant decls.
  • Removed unnecessary clang:: namespace for variables.
  • Executed arc lint
  • Final changes.
  • Executed arc lint
Oct 31 2020, 5:45 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Oct 30 2020

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Reply code review suggestions. I will upload my change later.

Oct 30 2020, 7:49 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Oct 23 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Changed the type of HNOption from std::shared_ptr to reference.
Oct 23 2020, 7:52 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Thank you for the suggestion.

Oct 23 2020, 6:25 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Oct 21 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Changed Hungarian Notation as an prefix style (not a case type).
  • Changed XxxHungarianPrefix as an enumeration from bool.
  • Updated documents.
Oct 21 2020, 7:45 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Oct 20 2020

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Hi @njames93,

It's a smart idea, the rework for it is worth. There is a special case if lowercase name with Hungarian prefix, it possibly makes variable ambiguous, like the Case1. How about separating them and aNy_CasE with an underscore, like Case2 ?

// Case1
bool bRIGHT_LEVEL;     // UPPER_CASE
bool bRightLevel;      // CamelCase
bool bRight_Level;     // Camel_Snake_Case
bool baNy_CasE;        // aNy_CasE
bool bright_level;     // lower_case
bool brightLevel;      // camelBack
bool bright_Level;     // camel_Snake_Back
.....^^^^^^ <-- right? bright?

// Case2
bool bRIGHT_LEVEL;     // UPPER_CASE
bool bRightLevel;      // CamelCase
bool bRight_Level;     // Camel_Snake_Case
bool b_aNy_CasE;       // aNy_CasE
bool b_right_level;    // lower_case
bool b_rightLevel;     // camelBack
bool b_right_Level;    // camel_Snake_Back
.....^^^^^^^ <-- add an underscore

That still has hidden surprises. Maybe instead of a bool, an enum is used for controlling hungarian prefix (Off|On|...).
Can't think of a good name for the third option but it would do the inserting of '_' (bright_level ->b_right_level) or capitalising the first word of the identifier (brightLevel -> bRightLevel).

Oct 20 2020, 4:38 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Is this diff been created incorrectly again?

Taking a step back, Is Hungarian notation really a case style, Seems to me its mainly about the prefix and a user may want DWORD dwUPPER_CASE, Right now there is no way of adopting that.
Maybe extend the options for hungarian type decls to

<Type>Case
<Type>Prefix
<Type>Suffix
<Type>HungarianPrefix

<Type>HungarianPrefix would likely be a bool and if enabled, it would be prepended to <Type>Prefix
I could see a situation like this

[Options]
// VariableCase: UPPER_CASE
// VariablePrefix: PRE_
// VariableSuffix: _POST
// VariableHungarianPrefix: true

DWORD MyVariable; -> DWORD dwPRE_MY_VARIABLE_POST;

This would give users full control of exactly how they want their declarations with no hidden surprises.

Granted this approach would require a little rework but it would satisfy more users.

Oct 20 2020, 7:54 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Oct 13 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Merged all commits as one single commit. (Trying to resolve "No such file or directory" error from the scripts/phabtalk/apply_patch2.py!_apply_diff() function)

Oct 13 2020, 10:47 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Hi @aaron.ballman and @njames93,
I addressed your code review suggestions and supported Hungarian Notation prefix for decl of enum and class(option) at latest patch. Unfortunately, I encountered a problem that new patch failed on remote BuildBot, it showed the "No such file or directory" error message when it was trying to call apply_patch2.py!_apply_diff(). Do you have any idea what is going on? Do you suggest I create a new Diff(new diff id) for it ?

Oct 13 2020, 6:20 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Rename file name, "readability-identifier-naming-hungarian-notation-default.cpp" to "readability-identifier-naming-hungarian-notation.cpp".

Oct 13 2020, 5:45 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Merged with the master then do arc diff master --update D86671 again. (Trying to resolve "No such file or directory" error from the scripts/phabtalk/apply_patch2.py!_apply_diff() function)

Oct 13 2020, 4:09 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Oct 11 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Previous arc diff got problem(failed on builtbot), do it again with arc diff master --update D86671.

Oct 11 2020, 6:40 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Oct 10 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Support to add Class prefix for Hungarian Notation.
  • Support to add Enum prefix for Hungarian Notation.
  • Support unsigned long long, ULONG, and HANDLE types and others.
  • Support options for Hungarian Notation in config file.
  • Added more test cases.
Oct 10 2020, 11:11 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added inline comments to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
Oct 10 2020, 9:46 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Sep 30 2020

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Not strictly relevant here, but this does open up the idea of enforcing the style where an enum constant is prefixed by the initials of the enum name.

Sep 30 2020, 8:17 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Hi @aaron.ballman
About changing size_t nLength to cbLength. I searched MSVC folders with size_t, many names of variable start with n, or i in MFC related files. So I prefer to keep it starts with n. Another side to name starts with cb, I found variables like cbXxx are defined with ULONG, DWORD, or UCHAR type.

I think the important point is that cb is used for APIs that specify a count of bytes, whereas i, n, and l are used for integers (there is no specific prefix for size_t that I'm aware of or that MSDN documents, so the lack of consistency there is not too surprising). That's why I mentioned that using a heuristic approach may allow you to identify the parameters that are intended to be a count of bytes so that you can use the more specific prefix if there's more specific contextual information available.

Sep 30 2020, 9:57 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Sep 20 2020

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Hi @aaron.ballman
About changing size_t nLength to cbLength. I searched MSVC folders with size_t, many names of variable start with n, or i in MFC related files. So I prefer to keep it starts with n. Another side to name starts with cb, I found variables like cbXxx are defined with ULONG, DWORD, or UCHAR type.

Sep 20 2020, 10:24 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Sep 19 2020

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Replied comments by @aaron.ballman

Sep 19 2020, 10:16 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Sep 13 2020

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Did you upload this incorrectly again, context is missing and seems to be a relative diff from a previous version of this patch?

Sorry for it, I think it's my bad. It is possible that I manually merged the last master(github) with changes then updated them both via web interface ...

Can I fix it if switch back to the base (14948a0) then merge all my changes, then update the diff again via web interface? Or do you have any better suggestion?

I am curious about how do you know this mistake? You got error messages with arc patch D86671 ?

The no context is easy to spot as phab says context not available. Its easy to find knowing that there is no mention of hungarian notation in the trunk version of IdentifierNamingCheck.cpp, yet there is mention of that in the before diff.

The way I do my patches is I create a branch from the current master. Then all commits go into that branch. When its time to update the PR I can just do a diff from <feature_branch> to <master>.
Though I do use arcanist for my patches

arc diff master

arcanist will check to see if the current branch has tags for PR and automatically update that PR. Otherwise it will create a new PR.
If it goes to create a new PR instead of updating an existing one you can pass update

arc diff master --update D86671
Sep 13 2020, 8:02 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Sep 11 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Fixed crash on Windows when run regression test (llvm-lit for readability-identifier-naming.cpp file).

Sep 11 2020, 7:53 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Sep 9 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
  • Fixed lint warnings and regression test failures on Windows.
Sep 9 2020, 6:43 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Sep 8 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

This is a test with arc diff master --update D86671 command.

Sep 8 2020, 7:30 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Sep 7 2020

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Did you upload this incorrectly again, context is missing and seems to be a relative diff from a previous version of this patch?

Sep 7 2020, 9:28 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Sep 5 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Addressed comments by @aaron.ballman

Sep 5 2020, 10:03 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Sep 2 2020

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Please no worry to give me your suggestions and feedback.

Sep 2 2020, 8:24 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Sep 1 2020

dougpuob added a comment to D86930: [clang-format] Handle typename macros inside cast expressions.

I am a beginner to compiler, interesting in how to write Unit Test case for change so I ran it, but found difference with my expection.

Sep 1 2020, 5:31 PM · Restricted Project

Aug 31 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Improved suggestions of code review.

  1. Moved release notes to right place. [Eugene.Zelenko]
  2. Added new casting types to doc(readability-identifier-naming.rst) [Eugene.Zelenko]
  3. Moved partial code to a new function, IdentifierNamingCheck::getDeclTypeName(). [njames93]
Aug 31 2020, 6:52 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Aug 30 2020

dougpuob added inline comments to D86847: [Bitcode] Add BITCODE_SIZE_BLOCK_ID to encode the size of the bitcode.
Aug 30 2020, 10:29 PM · Unknown Object (Project)
dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

As Hungarian notation enforces prefixes as well as casing styles it would be wise to warn on cases where the style is Hungarian but a prefix has also been set.

Another issue is this current set up will let you define any decls style as hungarian, this doesn't make sense for most decl types.
Potentially some validation should happen for that too,

Aug 30 2020, 10:24 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added inline comments to D86847: [Bitcode] Add BITCODE_SIZE_BLOCK_ID to encode the size of the bitcode.
Aug 30 2020, 12:07 AM · Unknown Object (Project)

Aug 29 2020

dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

By the word, you must mention new option in documentation too.

Aug 29 2020, 9:14 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added inline comments to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
Aug 29 2020, 8:39 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

I fixed all review suggestions from @aaron.ballman, @Eugene.Zelenko, and @njames93, Thank you. But I still can't sure I did everything correct.

Aug 29 2020, 2:08 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Improved for suggestions of code review.

Aug 29 2020, 1:48 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Aug 28 2020

dougpuob added inline comments to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
Aug 28 2020, 9:21 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added inline comments to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
Aug 28 2020, 9:14 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

It'll be good idea to add test case.

Aug 28 2020, 4:30 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob added a comment to D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Can you make sure you upload diffs with full context (-U99999). Or using arcanist it will be done automatically.

Make sure the diff is up to date with trunk

Remove any changes that aren't related to this patch, they just make this look noisy.

Aug 28 2020, 4:21 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Aug 27 2020

dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Improved suggestions from code review and clang-tidy.

  1. Add keyword const to variables which checked via clang-tidy.
  2. Add log to clang-tools-extra/docs/ReleaseNotes.rst. [Suggestion from Eugene.Zelenko]
  3. Don't use auto with variables are not specified explicitly. [Suggestion from Eugene.Zelenko]
Aug 27 2020, 8:06 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob updated the diff for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.

Fixed typos and add new Case Type, szHungarianNotation in doc.

Aug 27 2020, 6:02 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob retitled D86671: [clang-tidy] Add new case type to check variables with Hungarian notation from Add new IdentifierNamingCheck::CaseType, CT_HungarianNotion, supporting naming check with Hungarian Notion. to [clang-tidy] Add new case type to check variables with Hungarian notation.
Aug 27 2020, 5:58 AM · Unknown Object (Project), Unknown Object (Project), Restricted Project

Aug 26 2020

dougpuob changed the visibility for D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
Aug 26 2020, 7:20 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project
dougpuob requested review of D86671: [clang-tidy] Add new case type to check variables with Hungarian notation.
Aug 26 2020, 7:00 PM · Unknown Object (Project), Unknown Object (Project), Restricted Project