2013-12-06 22:17:02 |
Hans Joachim Desserud |
bug |
|
|
added bug |
2013-12-06 22:17:02 |
Hans Joachim Desserud |
attachment added |
|
clang-warnings.txt https://bugs.launchpad.net/bugs/1258667/+attachment/3924532/+files/clang-warnings.txt |
|
2013-12-06 22:22:40 |
Hans Joachim Desserud |
description |
Clang/LLVM 3.4 is scheduled to be released soon, and the release candidate is packaged in Debian Unstable. I took Widelands for a spin, and while the 3.4 release doesn't seem to spot any additional warnings, we have some which should be looked into/fixed. Most of them concerns the same entity which are declared as structs and classes in different places of the code. (I noticed this a while back and since I knew we wanted to use classes throughout I worked a bit on branch for it. Unfortunately it fails to compile due to default private accesibility in classes and I wasn't really comfortable setting _everything_ to public as that might not have been the original intention. So someone else should probably look at that.)
Since bug 913369 was closed once we became warning-free, I've filed a separate bug for the ones which have been added since then. |
Clang/LLVM 3.4 is scheduled to be released soon, and the release candidate is packaged in Debian Unstable. I took Widelands for a spin, and while the 3.4 release doesn't seem to spot any additional warnings, we have some which should be looked into/fixed. Most of them concerns the same entity which are declared as structs and classes in different places of the code. (I noticed this a while back and since I knew we wanted to use classes throughout I worked a bit on branch for it. Unfortunately it fails to compile due to default private accesibility in classes and I wasn't really comfortable setting _everything_ to public as that might not have been the original intention. So someone else should probably look at that.)
Since bug 913369 was closed once we became warning-free, I've filed a separate bug for the ones which have been added since then.
PS. For other warning reports, see bug 1202101 (Visual Studio) and bug 986611 (cppcheck). |
|
2013-12-06 22:26:55 |
Hans Joachim Desserud |
description |
Clang/LLVM 3.4 is scheduled to be released soon, and the release candidate is packaged in Debian Unstable. I took Widelands for a spin, and while the 3.4 release doesn't seem to spot any additional warnings, we have some which should be looked into/fixed. Most of them concerns the same entity which are declared as structs and classes in different places of the code. (I noticed this a while back and since I knew we wanted to use classes throughout I worked a bit on branch for it. Unfortunately it fails to compile due to default private accesibility in classes and I wasn't really comfortable setting _everything_ to public as that might not have been the original intention. So someone else should probably look at that.)
Since bug 913369 was closed once we became warning-free, I've filed a separate bug for the ones which have been added since then.
PS. For other warning reports, see bug 1202101 (Visual Studio) and bug 986611 (cppcheck). |
Clang/LLVM 3.4 is scheduled to be released soon, and the release candidate is packaged in Debian Unstable. I took Widelands* for a spin, and while the 3.4 release doesn't seem to spot any additional warnings, we have some which should be looked into/fixed. Most of them concerns the same entity which are declared as structs and classes in different places of the code. (I noticed this a while back and since I knew we wanted to use classes throughout I worked a bit on branch for it. Unfortunately it fails to compile due to default private accesibility in classes and I wasn't really comfortable setting _everything_ to public as that might not have been the original intention. So someone else should probably look at that.)
Since bug 913369 was closed once we became warning-free, I've filed a separate bug for the ones which have been added since then.
*latest trunk at the time was r6820
PS. For other warning reports, see bug 1202101 (Visual Studio) and bug 986611 (cppcheck). |
|
2014-04-21 11:44:15 |
Hans Joachim Desserud |
description |
Clang/LLVM 3.4 is scheduled to be released soon, and the release candidate is packaged in Debian Unstable. I took Widelands* for a spin, and while the 3.4 release doesn't seem to spot any additional warnings, we have some which should be looked into/fixed. Most of them concerns the same entity which are declared as structs and classes in different places of the code. (I noticed this a while back and since I knew we wanted to use classes throughout I worked a bit on branch for it. Unfortunately it fails to compile due to default private accesibility in classes and I wasn't really comfortable setting _everything_ to public as that might not have been the original intention. So someone else should probably look at that.)
Since bug 913369 was closed once we became warning-free, I've filed a separate bug for the ones which have been added since then.
*latest trunk at the time was r6820
PS. For other warning reports, see bug 1202101 (Visual Studio) and bug 986611 (cppcheck). |
Clang/LLVM 3.4 is scheduled to be released soon, and the release candidate is packaged in Debian Unstable. I took Widelands* for a spin, and while the 3.4 release doesn't seem to spot any additional warnings, we have some which should be looked into/fixed. Most of them concerns the same entity which are declared as structs and classes in different places of the code. (I noticed this a while back and since I knew we wanted to use classes throughout I worked a bit on branch for it. Unfortunately it fails to compile due to default private accesibility in classes and I wasn't really comfortable setting _everything_ to public as that might not have been the original intention. So someone else should probably look at that.)
Since bug 913369 was closed once we became warning-free, I've filed a separate bug for the ones which have been added since then.
*latest trunk at the time was r6820
PS. For other warning reports, see bug 986611 (cppcheck), bug 1278174 (flawfinder) and bug 1202101 (Visual Studio). |
|
2014-09-05 13:12:23 |
SirVer |
widelands: status |
Triaged |
Incomplete |
|
2014-09-13 07:08:49 |
Hans Joachim Desserud |
attachment added |
|
Clang 3.5 vs r7175 https://bugs.launchpad.net/widelands/+bug/1258667/+attachment/4202898/+files/widelands-clang.new |
|
2014-09-13 07:09:03 |
Hans Joachim Desserud |
summary |
Compilation warnings with Clang (3.4) |
Compilation warnings with Clang (3.5) |
|
2014-09-13 07:09:08 |
Hans Joachim Desserud |
widelands: milestone |
build19-rc1 |
|
|
2014-09-13 07:09:15 |
Hans Joachim Desserud |
widelands: status |
Incomplete |
Confirmed |
|
2017-08-12 08:57:58 |
GunChleoc |
widelands: status |
Confirmed |
Fix Committed |
|
2017-08-12 08:58:00 |
GunChleoc |
widelands: milestone |
|
build20-rc1 |
|
2019-04-22 07:01:17 |
GunChleoc |
widelands: status |
Fix Committed |
Fix Released |
|