[Github-comments] [geany/geany-plugins] scope: small code cleanup. See #794. (#795)

[Colomban Wendling]

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.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/795#issuecomment-443164557
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20181130/e3e7f739/attachment.html>