Hello all,<\/p>\n
I’m here with my week 3 report for Modern C porting of Gentoo’s
\npackages. For this week I diverted from my initial idea a bit and
\nfocused on the “C++17 does not allow register storage class specific” type
\nerror. Basically, C++14 deprecated the register storage class and it has
\nbeen completely removed in C++17, thus resulting in C++ packages that
\nuse register keywords with this kind of error. A general fix is it
\neither removes the keywords or replaces them with *int* where applicable.<\/p>\n
For example, in this PR [1] for the fox toolkit, I’m using sed to remove
\nregister keywords from various folders of the source. Whenever possible
\nI’m sending patches upstream as well, for example, I’ve sent this [2]
\npatch upstream while also applying it Gentoo tree.<\/p>\n
Not to mention, I’ve been sticking to my proposal timeline as well and
\npatching -Wimplicit-int error [3][4][5] (Initially I planned on taking
\nup this particular bug type in the latter half of the week). But
\nsometimes the error messages are not that straightforward and can be
\nmiss leading. For example in this PR [3], although the bugs say that
\nit might be a -Wimplicit-int bug it was rather just a wrong inclusion of
\nheader where the data type off_t was not being found and it was assumed
\nto be the parameter.<\/p>\n
Next:
\nAs per my proposal, I’m going to stick to -Wimplicit-int for the rest few
\ndays of the week and then slowly move to
\n-Wimplicit-function-declaration. I did come across some
\n-Wimplicit-function-declaration type of bugs which I’ve patched and
\ncreated a PR [6][7].<\/p>\n
I’ve also come across two main language-specific blockers with
\nclang-16, namely Fortran and Vala. Lots of packages from sci-libs are
\ndependent on the package sci-libs\/lapack which is failing to build on
\nwith clang-16. As a result, I’ve to keep some of the sci-libs bugs on
\nhalt for now. On the Vala front, not much we (as in Gentoo people\/devs)
\ncan do, this is a main blocker for some GNOME packages.<\/p>\n
That’s it for this week, hopefully, I’ll be able to patch more packages
\nin the coming weeks.<\/p>\n
[1]: https:\/\/github.com\/gentoo\/gentoo\/pull\/31357\/files#diff-aaf358e6aacd565fcdd354d1ff87b08b1b2c679aaf978af900caa00c70a7978eR46<\/a> Hello all, I’m here with my week 3 report for Modern C porting of Gentoo’s packages. For this week I diverted from my initial idea a bit and focused on the “C++17 does not allow register storage class specific” type … Continue reading
\n[2]: https:\/\/github.com\/VirtualGL\/virtualgl\/commit\/441c4e77d8e33edb28d3015b573e4e45bb13d684<\/a>
\n[3]: https:\/\/github.com\/gentoo\/gentoo\/pull\/31520<\/a>
\n[4]: https:\/\/github.com\/gentoo\/gentoo\/pull\/31464<\/a>
\n[5]: https:\/\/github.com\/gentoo\/gentoo\/pull\/31411<\/a>
\n[6]: https:\/\/github.com\/gentoo\/gentoo\/pull\/31476<\/a>
\n[7]: https:\/\/github.com\/gentoo\/gentoo\/pull\/31513<\/a>
\n<\/span><\/p>\n","protected":false},"excerpt":{"rendered":"