Jesse Vincent
a0db8238ff
astyle and lint
8 years ago
Jesse Vincent
d77b1a5c41
make astyle
8 years ago
Gergely Nagy
101da6450f
Major documentation update
...
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Jesse Vincent
5609a40db3
revert this change. it doesn't pass the linter, but it does actually
...
work
8 years ago
Jesse Vincent
2d87a3888f
fixups from astyling and linting
8 years ago
Gergely Nagy
8220369f6f
Kaleidoscope Style Guide conformance
...
Also updated to work with the newest `EEPROM-Keymap` API.
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Gergely Nagy
3b4e2add7d
Conform to the latest Kaleidoscope Style Guide
...
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Jesse Vincent
5215e95249
make astyle
8 years ago
Gergely Nagy
60882c0f1e
README.md: Document the Focus commands too
...
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Gergely Nagy
cfd80f3f69
Conform to the latest Kaleidoscope Style Guide
...
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Jesse Vincent
8c6b8433cc
astyle pass
8 years ago
Jesse Vincent
57ebb17d8f
Revert "switch from one space indent to two space indent before public: and private:"
...
This reverts commit 502bc0b59f
.
Now that we have two space indent, astyle does the right thing
8 years ago
Gergely Nagy
8782cea7c1
Focus: When printing the keymap, use the effective layout
...
Instead of printing the keymap from EEPROM, print the effective layout.
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Gergely Nagy
ba6128dddc
Kaleidoscope Style Guide conformance
...
Updated the code to conform to the latest style guide. And added a bit of
documentation too, while there.
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Jesse Vincent
88fc70f90a
Update whitespace per new astyle rules
8 years ago
Jesse Vincent
502bc0b59f
switch from one space indent to two space indent before public: and private:
...
This matches what astyle does, so it's one less thing to stress about
8 years ago
Gergely Nagy
38cf866870
Kaleidoscope Style Guide conformance
...
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Gergely Nagy
b8f47755ab
Yet more styling fixes
...
Put the return type and the function name on the same line in the .cpp file too.
While neither astyle, nor cpplint warn about this, this is how Kaleidoscope core
does it, so lets do the same.
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Jesse Vincent
fcff6f3d20
Make the linter happy
8 years ago
Jesse Vincent
a497d67267
Make the linter happy
8 years ago
Gergely Nagy
66c8a19f21
Re-run make astyle, with a recent astyle
...
Use astyle 3.0 (included in Kaleidoscope-Plugin) to apply styling, instead of
whatever came with Debian Stretch.
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Jesse Vincent
9129b8e9a7
Switch up the -libraries order so the local version of a library gets
...
found first
8 years ago
Gergely Nagy
70e4d15e60
More Style Guide fixes
...
Drop the namespace terminating comment, it is not neccessary, and run astyle
again.
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Jesse Vincent
1d8c3040f1
Fix issues found by the linter.
8 years ago
Jesse Vincent
1d47c216aa
whitespace fix
8 years ago
Jesse Vincent
a49b987d5c
Make the linter happy
8 years ago
Gergely Nagy
3e982ba5f6
Kaleidoscope Style Guide conformance
...
Rearranged both in style, and in naming conventions to match the Kaleidoscope
Style Guide, and please the linter too.
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Jesse Vincent
2e3b93d03d
Changes to make the linter happy
8 years ago
Jesse Vincent
a02fc71a3b
add some comments to make the linter happy
8 years ago
Jesse Vincent
7c1124af2c
remove a virtual demanded by the linter
8 years ago
Jesse Vincent
6138e2f5be
Remove unneeded ;
8 years ago
Gergely Nagy
522fcc8c25
doc/CODING_STYLE.html: Fix a small typo
...
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Jesse Vincent
e57cfbc0db
remove a bunch of semicolons that the linter says are superfluous
8 years ago
Jesse Vincent
02b144cff3
Switch to cpplint 1.30 from https://pypi.python.org/pypi/cpplint
...
Unlike the upstream google linter, it understands '#pragma once'
8 years ago
Jesse Vincent
37d0f1cb8c
Switch HIDTables.h to #pragma once to satisfy the linter.
...
This _might_ get us in trouble with duplicate definitions in KeyboardioHID
8 years ago
Jesse Vincent
854a7c73b2
Import google cpplint
8 years ago
Jesse Vincent
554b43c2b5
Remove an unneeded ; caught by cpplint.py
8 years ago
Jesse Vincent
4b27f711f9
Strike through windows-specific stuff
8 years ago
Jesse Vincent
cf4886f653
Add a note about access control. Remove a google-specific note
8 years ago
Jesse Vincent
44d95f050e
We use astyle
8 years ago
Jesse Vincent
7cd1eee07d
our guidance on inline functions is a little bit difference because we're targeting embedded devices
8 years ago
Jesse Vincent
02ea090ac7
We use #pragma once
8 years ago
Jesse Vincent
01bfee48cb
GCC extensions are ok
8 years ago
Jesse Vincent
5873b461ac
we don't use std::hash
8 years ago
Jesse Vincent
b0577d241c
Boost does not make sense in this context
8 years ago
Jesse Vincent
30edd7da2f
64 bit portability is a non-goal
8 years ago
Jesse Vincent
ce595a848e
Switch the style guide to HTML, rather than wasting hours butchering
...
Google's nice semantic HTML to Markdown
8 years ago
Jesse Vincent
f7b071d134
Add notes from @obra and @algernon. As yet not integrated
8 years ago
Jesse Vincent
b33669af45
Checking in a markdown-ified (but otherwise unmodified) version of the
...
Google C++ style guide as of June 2, 2017
8 years ago
Gergely Nagy
09ba7dd756
Port over to LED-Palette-Theme
...
Fixes #6 .
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago