C style casts

Bug #717279 reported by Olaf van der Spek on 2011-02-11
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Drizzle
Wishlist
Unassigned

Bug Description

C casts shouldn't be used in C++, so -Wold-style-cast might be handy.

Related branches

Monty Taylor (mordred) wrote :

Agree... and I welcome patches to fix the tree in this manner. For now, I suggest:

make CPPFLAGS='-Wold-style-cast'

To trigger places to find and fix. We can't add it to the official list of warnings yet, as there are THOUSANDS of them.

On Fri, 11 Feb 2011 19:26:43 -0000, Monty Taylor <email address hidden> wrote:
> Agree... and I welcome patches to fix the tree in this manner. For now,
> I suggest:
>
> make CPPFLAGS='-Wold-style-cast'
>
> To trigger places to find and fix. We can't add it to the official list
> of warnings yet, as there are THOUSANDS of them.

do we have a Jenkins target with all these nice-to-haves on and the
warnings tracker thing on?

--
Stewart Smith

Monty Taylor (mordred) wrote :

Yup. That would be drizzle-build-all-warnings

"Stewart Smith" <email address hidden> wrote:

>On Fri, 11 Feb 2011 19:26:43 -0000, Monty Taylor <email address hidden>
>wrote:
>> Agree... and I welcome patches to fix the tree in this manner. For
>now,
>> I suggest:
>>
>> make CPPFLAGS='-Wold-style-cast'
>>
>> To trigger places to find and fix. We can't add it to the official
>list
>> of warnings yet, as there are THOUSANDS of them.
>
>do we have a Jenkins target with all these nice-to-haves on and the
>warnings tracker thing on?
>
>--
>Stewart Smith
>
>--
>You received this bug notification because you are a member of Drizzle-
>developers, which is subscribed to Drizzle.
>https://bugs.launchpad.net/bugs/717279
>
>Title:
> C style casts
>
>Status in A Lightweight SQL Database for Cloud Infrastructure and Web
>Applications:
> New
>
>Bug description:
> C casts shouldn't be used in C++, so -Wold-style-cast might be handy.

--
Sent from my Android phone with K-9 Mail. Please excuse my brevity.

Lee Bieber (kalebral) on 2011-02-21
Changed in drizzle:
milestone: none → future
importance: Undecided → Low
status: New → Confirmed
importance: Low → Wishlist
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers