Review Request 127968: kshareddatacache: Fix invalid use of & to avoid unaligned reads

[Fabian Vogt]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127968/
-----------------------------------------------------------

Review request for KDE Frameworks, David Faure and Michael Pyne.


Repository: kcoreaddons


Description
-------

That the & is wrong here is obvious:
align can never be 2 inside the if condition as ```(2 & bool)``` is always false,
yet the switch on align contains a case for 2.
Instead it drops into the while loop for an 4-byte aligned pointer.

This doesn't change the returned values on platforms where unaligned reads
are permitted, so I didn't bump the version.


Diffs
-----

  src/lib/caching/kshareddatacache.cpp 50bbf64 

Diff: https://git.reviewboard.kde.org/r/127968/diff/


Testing
-------

Ran make test, all passed.


Thanks,

Fabian Vogt

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160519/fdf1002d/attachment-0001.html>