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
88fc70f90a
Update whitespace per new astyle rules
8 years ago
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
38cf866870
Kaleidoscope Style Guide conformance
...
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
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
fcff6f3d20
Make the linter happy
8 years ago
a497d67267
Make the linter happy
8 years ago
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
9129b8e9a7
Switch up the -libraries order so the local version of a library gets
...
found first
8 years ago
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
1d8c3040f1
Fix issues found by the linter.
8 years ago
1d47c216aa
whitespace fix
8 years ago
a49b987d5c
Make the linter happy
8 years ago
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
2e3b93d03d
Changes to make the linter happy
8 years ago
a02fc71a3b
add some comments to make the linter happy
8 years ago
7c1124af2c
remove a virtual demanded by the linter
8 years ago
6138e2f5be
Remove unneeded ;
8 years ago
522fcc8c25
doc/CODING_STYLE.html: Fix a small typo
...
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
e57cfbc0db
remove a bunch of semicolons that the linter says are superfluous
8 years ago
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
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
854a7c73b2
Import google cpplint
8 years ago
554b43c2b5
Remove an unneeded ; caught by cpplint.py
8 years ago
4b27f711f9
Strike through windows-specific stuff
8 years ago
cf4886f653
Add a note about access control. Remove a google-specific note
8 years ago
44d95f050e
We use astyle
8 years ago
7cd1eee07d
our guidance on inline functions is a little bit difference because we're targeting embedded devices
8 years ago
02ea090ac7
We use #pragma once
8 years ago
01bfee48cb
GCC extensions are ok
8 years ago
5873b461ac
we don't use std::hash
8 years ago
b0577d241c
Boost does not make sense in this context
8 years ago
30edd7da2f
64 bit portability is a non-goal
8 years ago
ce595a848e
Switch the style guide to HTML, rather than wasting hours butchering
...
Google's nice semantic HTML to Markdown
8 years ago
f7b071d134
Add notes from @obra and @algernon. As yet not integrated
8 years ago
b33669af45
Checking in a markdown-ified (but otherwise unmodified) version of the
...
Google C++ style guide as of June 2, 2017
8 years ago
09ba7dd756
Port over to LED-Palette-Theme
...
Fixes #6 .
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
915e7068f6
Merge pull request #7 from keyboardio/f/get-led-index
...
Add a `get_led_index` function
8 years ago
733511fc37
Add a `get_led_index` function
...
Given a `row`/`col` coordinate, this new function returns the index of the LED
under that key. To be used with the `led_set_crgb_at(i, crgb)` function.
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
0fc32dbd94
Add a number of helper functions
...
Added `lookupColor(index)`, which looks up a color in the palette, by index.
Made the `row, col` variant public, and added an `updateColor` method.
These are there to help porting FingerPainter.
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
9e10c9e686
make astyle
...
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
947019733d
Rename to Kaleidoscope-Colormap
...
Fixes #2 .
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
34c5297875
Only request a palette slice, if we haven't already
...
This should - in theory - allow other plugins to just use `LEDPaletteTheme`,
without having many copies of the palette.
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
7b8ae3c9c0
examples: Fix the example build
...
Include Focus, so the example will build.
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
f7196ef26d
Migrate to LED-Palette-Theme
...
Fixes #3 .
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
b81cc12708
Initial import
...
Fixes #1 .
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
813a48a04f
Add the ability to specify CFLAGS through the builder
8 years ago
356dd681f9
astyle 3.0 fixups
8 years ago
ede5fdba6c
astyle with astyle 3.0, which matters for this code
8 years ago
437a349288
make astyle
8 years ago
Gergely Nagy
Jesse Vincent
Jesse Vincent