D14479: Store sizeof and friends as numbers instead of comment.
Michal Srb
noreply at phabricator.kde.org
Mon Aug 13 16:20:28 BST 2018
michalsrb added inline comments.
INLINE COMMENTS
> brauch wrote in classmemberdeclaration.h:75
> this comment is wrong, it returns the size, not the alignment
copy paste mistake, I will fix that
> brauch wrote in classmemberdeclaration.h:77
> Why is this a 64-bit integer? I.e. why is 32-bit not sufficient?
I chose that because that is what `clang_Type_getSizeOf` returns. You can declare something like `char c[0x100000000]` and it will display the size properly.
But I expect that almost no real code will have sizes or offsets this big, so we could use 32-bit integer instead and fallback to -1 for the extreme cases. Do you think we should?
REPOSITORY
R32 KDevelop
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D14479
To: michalsrb, #kdevelop, aaronpuchert
Cc: brauch, aaronpuchert, kdevelop-devel, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20180813/28129fc3/attachment-0001.html>
More information about the KDevelop-devel
mailing list