Fixed ```sizeof``` without parentheses, spaces instead of tab and a compiler warning. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany-plugins/pull/795
-- Commit Summary --
* scope: small code cleanup. See #794.
-- File Changes --
M scope/src/conterm.c (8) M scope/src/memory.c (3) M scope/src/utils.c (4)
-- Patch Links --
https://github.com/geany/geany-plugins/pull/795.patch https://github.com/geany/geany-plugins/pull/795.diff
As said on #794 there was nothing wrong in those `sizeof` expressions, if anything they weren't as readable as possible; and the real issue is a bug in cppcheck.
This said, the changes here look fine, and will indeed work around that cppcheck issue, and IMO makes that `pointer_size` logic a little clearer. So, LGTM.
Merged #795 into master.
github-comments@lists.geany.org