Kleine 8x8 wortuhr #483
Merged
Kleine 8x8 wortuhr #483
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Betrifft den issue #480, ich weiß aber nicht, wie ich den verlinke. |
|
Vielen Dank für deinen Pullrequest, ich schaue mir das mal an, wenn ich morgen mal wieder etwas Zeit habe :-) die letzten Wochen waren echt etwas viel. Mit freundlichen Grüßen |
|
Kein Ding, sind ja auch eine Menge Änderungen und Dezember ist immer etwas stressig... Ich bin auch noch nicht dazugekommen, die Spanische und die Deutsche Variante mit 5-Minuten einzupflegen. |
…layout-ESP-Wordclock into Kleine-8x8-Wortuhr
…layout-ESP-Wordclock into Kleine-8x8-Wortuhr
…layout-ESP-Wordclock into Kleine-8x8-Wortuhr
|
Hey @BastiMu Sag einfach an, sobald es fertig und getestet ist. Dann merge ich das :-) Mit freundlichen Grüßen |
…layout-ESP-Wordclock into Kleine-8x8-Wortuhr
hoffie
added a commit
to hoffie/Multilayout-ESP-Wordclock
that referenced
this pull request
Apr 6, 2025
PR ESPWortuhr#483 introduced changes which reference the fontWidth and fontHeight variables directly as arrays. However, those two variables are declared as PROGMEM and may therefore only be referenced as addresses which will be read from flash via pgm_read_byte. The array access seems to cause crashes in certain configurations (and maybe undefined behavior in others?). This commit changes all fontWidth and fontHeight array accesses to be in line with the previous usage of those variables: The data is read via pgm_read_byte. Fixes: ESPWortuhr#557 Related: ESPWortuhr#550 (comment) Signed-off-by: Christian Hoffmann <[email protected]>
hoffie
reviewed
Apr 6, 2025
| bool showHours = true; | ||
| bool showMinutes = true; | ||
| // toogle hours and minutes if clock is not high enough | ||
| if (usedUhrType->rowsWordMatrix() < (fontHeight[usedFontSize] * 2 + 1)) { |
There was a problem hiding this comment.
Nope. This is an error.
The toggling is only necessary for the 8x8
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Die 8x8 Wortuhr ist Aufgrund der Größe auf viertel Stunden limitiert. Außerdem funktionieren durch die kleine Matrix nicht mehr alle Funktionen identisch. Daher waren einige Änderungen notwendig, die ich hier kurz aufliste:
Ich habe die Wortuhr 8x8 Viertelstunde genannt, da ich später festgestellt habe, das man im Deutschen durch FÜNFZEHN statt VIERTEL auch 5-Minuten-weise die Zeit anzeigen kann. Diese will ich noch erstellen, die viertelstunden Einstellungen benötige ich jedoch auch, da auch eine Spanische Version geplant ist. Außerdem plane ich eine große 13x13 Wortuhr mit allen Minuten (daher der has60MinuteLayout identifier).