Skip to content
This repository has been archived by the owner on Jul 13, 2019. It is now read-only.

Fix wrong report for "public slots" #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix wrong report for "public slots" #6

wants to merge 1 commit into from

Conversation

tkruse
Copy link
Owner

@tkruse tkruse commented Apr 23, 2016

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
Copy link
Owner Author

tkruse commented Apr 23, 2016

also see #8

@tkruse tkruse force-pushed the multi-word branch 6 times, most recently from dc24cad to 78587cc Compare April 24, 2016 15:03
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 that referenced this pull request May 12, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant