clarify confusing warning

Bug #1766732 reported by Devon Sean McCullough
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
CFFI
New
Undecided
Unassigned

Bug Description

The unpatched warning leads the reader to assume that
 (1) the warning refers to struct type values when in fact it refers to struct type names, and (2) therefore the literal code in the message is intended in some abstract, figurative sense.

Revision history for this message
Devon Sean McCullough (devon7) wrote :

Oops, ignore the description and use the attachment.

Revision history for this message
Devon Sean McCullough (devon7) wrote :

Argh, isn't there some way to edit a bug report? Use patch#3.

description: updated
Revision history for this message
Devon Sean McCullough (devon7) wrote :

Hmm, no way to delete my now nonsensical comments?

Revision history for this message
Luís Oliveira (luismbo) wrote :

Why do you say the warning refers to values?

Revision history for this message
Devon Sean McCullough (devon7) wrote :

Why should the warning use the same language as the code itself?
Because the current warning is ambiguous and unclear.

Revision history for this message
Luís Oliveira (luismbo) wrote : Re: [Bug 1766732] Re: clarify confusing warning

I wonder if it's the verb "reference" that makes it unclear.

On Wed, Apr 25, 2018, 12:01 Devon Sean McCullough <
<email address hidden>> wrote:

> Why should the warning use the same language as the code itself?
> Because the current warning is ambiguous and unclear.
>
> --
> You received this bug notification because you are subscribed to CFFI.
> https://bugs.launchpad.net/bugs/1766732
>
> Title:
> clarify confusing warning
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/cffi/+bug/1766732/+subscriptions
>

Revision history for this message
Devon Sean McCullough (devon7) wrote :

[No way to amend nor delete even fresh unseen comments - huge flaw in this "launchpad" tool?]

old: bare references to struct types are deprecated
new: bare references to struct type names are deprecated
perhaps better yet: bare struct type names are deprecated

To my eye, the old warning "obviously" refers to struct type values.
I have bills to pay, work to do, I'd rather not have to dive down
some recursive rabbit hole, yak shaving just to fix some library
which happens to use CFFI which happens to spam my app console.
Clarifying said spam creates a friendly, pleasant experience,
a ray of light in the cold, dark, scary cave of C libraries.
Meanwhile, I'm dusting off plokami's two-year-old cobwebs.

Thanks for your prompt attention, I half expected a vacant lot here
as Ubuntu Launchpad is not mentioned in the docs, just in the code.

Revision history for this message
Devon Sean McCullough (devon7) wrote :

I agree, "references" misleads.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.