Skip to content
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.

Sign up for GitHub

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

Jump to bottom

cpplint can't handle multi word access specifiers (e.g., public slots:) correctly #30

Closed
GoogleCodeExporter opened this issue May 20, 2015 · 1 comment
Closed

cpplint can't handle multi word access specifiers (e.g., public slots:) correctly #30

GoogleCodeExporter opened this issue May 20, 2015 · 1 comment
Labels
auto-migrated cpplint

Comments

@GoogleCodeExporter
Copy link 

Source code: 

class X : public QObject {
...
 public slots:
...
}

causes incorrectly message "Weird number of spaces at line-start.  Are you 
using a 2-space indent?  [whitespace/indent] [3]". 

Attached patch removes message.

Original issue reported on code.google.com by [email protected] on 1 Oct 2014 at 9:09

Attachments:
tkruse added a commit to tkruse/cpplint that referenced this issue Apr 23, 2016
@tkruse
Fix wrong report for "public slots"
9d48107 
based on google/styleguide#30
original patch by [email protected]

Source code:

class X : public QObject {
...
 public slots:
...
}

causes incorrectly message "Weird number of spaces at line-start.  Are you
using a 2-space indent?  [whitespace/indent] [3]".
Signed-off-by: Thibault Kruse <[email protected]>
@tkruse tkruse mentioned this issue Apr 23, 2016
Open
tkruse added a commit to tkruse/cpplint that referenced this issue Apr 23, 2016
@tkruse
Fix wrong report for "public slots"
5f62ee4 
based on google/styleguide#30
original patch by [email protected]

Source code:

class X : public QObject {
...
 public slots:
...
}

causes incorrectly message "Weird number of spaces at line-start.  Are you
using a 2-space indent?  [whitespace/indent] [3]".
Signed-off-by: Thibault Kruse <[email protected]>
tkruse added a commit to tkruse/cpplint that referenced this issue Apr 23, 2016
@tkruse
Fix wrong report for "public slots"
46fedfe 
based on google/styleguide#30
original patch by [email protected]

Source code:

class X : public QObject {
...
 public slots:
...
}

causes incorrectly message "Weird number of spaces at line-start.  Are you
using a 2-space indent?  [whitespace/indent] [3]".
Signed-off-by: Thibault Kruse <[email protected]>
tkruse added a commit to tkruse/cpplint that referenced this issue Apr 23, 2016
@tkruse
Fix wrong report for "public slots"
1510601 
based on google/styleguide#30
original patch by [email protected]

Source code:

class X : public QObject {
...
 public slots:
...
}

causes incorrectly message "Weird number of spaces at line-start.  Are you
using a 2-space indent?  [whitespace/indent] [3]".
Signed-off-by: Thibault Kruse <[email protected]>
tkruse added a commit to tkruse/cpplint that referenced this issue Apr 23, 2016
@tkruse
Fix wrong report for "public slots"
22703eb 
based on google/styleguide#30
original patch by [email protected]

Source code:

class X : public QObject {
...
 public slots:
...
}

causes incorrectly message "Weird number of spaces at line-start.  Are you
using a 2-space indent?  [whitespace/indent] [3]".
Signed-off-by: Thibault Kruse <[email protected]>
tkruse added a commit to tkruse/cpplint that referenced this issue Apr 24, 2016
@tkruse
Fix wrong report for "public slots"
dc24cad 
based on google/styleguide#30
original patch by [email protected]

Source code:

class X : public QObject {
...
 public slots:
...
}

causes incorrectly message "Weird number of spaces at line-start.  Are you
using a 2-space indent?  [whitespace/indent] [3]".
Signed-off-by: Thibault Kruse <[email protected]>
tkruse pushed a commit to tkruse/cpplint that referenced this issue Apr 24, 2016
@tkruse
Fix wrong report for "public slots"
78587cc 
based on google/styleguide#30
original patch by [email protected]

Source code:

class X : public QObject {
...
 public slots:
...
}

causes incorrectly message "Weird number of spaces at line-start.  Are you
using a 2-space indent?  [whitespace/indent] [3]".
tkruse pushed a commit to tkruse/cpplint that referenced this issue Apr 24, 2016
@tkruse
Fix wrong report for "public slots"
21afb6c 
based on google/styleguide#30
original patch by [email protected]

Source code:

class X : public QObject {
...
 public slots:
...
}

causes incorrectly message "Weird number of spaces at line-start.  Are you
using a 2-space indent?  [whitespace/indent] [3]".
tkruse pushed a commit to tkruse/styleguide that referenced this issue May 13, 2016
@tkruse
Fix wrong report for "public slots"
7256a18 
based on google#30
original patch by [email protected]

Source code:

class X : public QObject {
...
 public slots:
...
}

causes incorrectly message "Weird number of spaces at line-start.  Are you
using a 2-space indent?  [whitespace/indent] [3]".
@tkruse tkruse mentioned this issue Jul 13, 2019
Closed
@vapier
Copy link
Member

vapier commented Jan 17, 2025

public cpplint development in this repository has been halted (see #837). please migrate to https://github.com/cpplint/cpplint now.

@vapier vapier closed this as not planned Won't fix, can't repro, duplicate, stale Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
auto-migrated cpplint
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vapier @GoogleCodeExporter