D21855: [IdUtils] Fix aliasing warning

Stefan Brüns noreply at phabricator.kde.org
Mon Jun 17 00:38:31 BST 2019


bruns added a comment.


  In D21855#480992 <https://phabricator.kde.org/D21855#480992>, @thiago wrote:
  
  > Alternative: return (quint64(devId) << 32) | inode;
  >
  > Not endian-dependent, like the memcpy solution. Make sure the other side is adapted to match.
  
  
  Thats the problem here - the "other side" is an ondisk DB ...
  
  But removing devId from the ID and just keeping inode is on the todo list, we are asking for trouble here anyway in case of 64bit inodes.

REPOSITORY
  R293 Baloo

REVISION DETAIL
  https://phabricator.kde.org/D21855

To: bruns, #baloo, ngraham, astippich, poboiko, thiago
Cc: kde-frameworks-devel, LeGast00n, fbampaloukas, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190616/9100061e/attachment.html>


More information about the Kde-frameworks-devel mailing list