-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Initializing member variable in class declaration masks variable name error #69
Comments
Variables inside the class should be m_[variable_Name] to represent it as the member variable. Close the issue please. Thanks |
Has this bug been fixed? If so, which was the pull request and merge?
…On Mon, Feb 24, 2020 at 2:47 AM Muhammad Wasim Akhtar Khan < ***@***.***> wrote:
Variables inside the class should be m_[variable_Name] to represent it as
the member variable. Close the issue please. Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#69?email_source=notifications&email_token=AACEJLZG4QJ6QJWI4JJQ46DREN3PHA5CNFSM4DL526GKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMW3ZJQ#issuecomment-590199974>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACEJL2YQRYLWPKLQWT6ANDREN3PHANCNFSM4DL526GA>
.
--
Stephen R. Aylward, Ph.D.
Senior Director of Strategic Initiatives
---
Kitware: *Advancing the frontiers of understanding*
*by developing innovative open-source software platforms*
*and integrating them into research, processes, and products.*
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
With
KWStyle.xml
:and
Test.h
:KWStyle fails to show the error in the name of the member variable.
Without the initialization of the variable in the header the error is detected correctly:
The error should still show even when the variable is initialized in the class declaration.
Tested with e03980f.
The text was updated successfully, but these errors were encountered: