Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Miscellaneous fixes #109

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ijackson
Copy link

While debugging a problem with button zones, I found these patches useful and/or necessary. They mostly stand alone.

(EE) BUG: triggered 'if (f[f_idx])'
(EE) BUG: ../../os/log.c:565 in vpnprintf()
(EE) Unsupported printf directive '+'
Otherwise we get an implicit declaration warning about mtdev_empty.

Signed-off-by: Ian Jackson <[email protected]>
Otherwise we get an implicit declaration warning about close().

Signed-off-by: Ian Jackson <[email protected]>
This is a printf format specifier, a la PRIx32 et al.  This avoids us
having to write %#x explicitly, which would be annoying if bitmask_t
ever changed its type.

No use sites introduced in this commit, so no functional change yet.

Signed-off-by: Ian Jackson <[email protected]>
The user is expected to decode the hex (with reference to the bit
numbers defined in mtstate.h).

Signed-off-by: Ian Jackson <[email protected]>
@ijackson ijackson changed the title For upstream.misc fixes Miscellaneous fixes Apr 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants