Skip to content

Remove Warnings - #401

Open
synthfi wants to merge 2 commits into
lneely:mainfrom
synthfi:remove-warnings
Open

Remove Warnings#401
synthfi wants to merge 2 commits into
lneely:mainfrom
synthfi:remove-warnings

Conversation

@synthfi

@synthfi synthfi commented Jul 13, 2026

Copy link
Copy Markdown

Changes

  • added a const qualifier in keyboard_buddies to resolve compilation warning
  • added a return 0 in daemonize to resolve a compilation warning
    • I am unsure if this is the correct thing to do

Notes

  • compiling w/ g++ 16.1.1 on fedora 44 aarch64
  • I saw this while running the tests, I don't know if this is a problem
=== tests/test_pcl26j_free ===
PASS: request_range: pmem_free called with header ptr, not data ptr
PASS: synced_down_folder: pmem_free called with header ptr, not data ptr
PASS: folder_tasks: pmem_free called with header ptr, not data ptr
PASS: sector_inlog: pmem_free called with header ptr, not data ptr
BAD FREE: 0x232b4a50 was not a malloc-returned pointer
PASS: harness self-check: bare free(data_ptr) correctly flagged

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.

1 participant