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
Jesse Vincent
915e7068f6
Merge pull request #7 from keyboardio/f/get-led-index
...
Add a `get_led_index` function
8 years ago
Gergely Nagy
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
Gergely Nagy
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
Gergely Nagy
9e10c9e686
make astyle
...
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Gergely Nagy
947019733d
Rename to Kaleidoscope-Colormap
...
Fixes #2 .
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Gergely Nagy
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
Gergely Nagy
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
Gergely Nagy
f7196ef26d
Migrate to LED-Palette-Theme
...
Fixes #3 .
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Gergely Nagy
b81cc12708
Initial import
...
Fixes #1 .
Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>
8 years ago
Jesse Vincent
813a48a04f
Add the ability to specify CFLAGS through the builder
8 years ago
Jesse Vincent
356dd681f9
astyle 3.0 fixups
8 years ago
Jesse Vincent
ede5fdba6c
astyle with astyle 3.0, which matters for this code
8 years ago
Jesse Vincent
437a349288
make astyle
8 years ago
Jesse Vincent
3b32197829
make astyle
8 years ago
Jesse Vincent
26e7e187df
make astyle
8 years ago
Jesse Vincent
a16411ff93
make astyle
8 years ago