• Open Menu Close Menu
  • Apple
  • Shopping Bag
  • Apple
  • Mac
  • iPad
  • iPhone
  • Watch
  • TV
  • Music
  • Support
  • Search apple.com
  • Shopping Bag

Lists

Open Menu Close Menu
  • Terms and Conditions
  • Lists hosted on this site
  • Email the Postmaster
  • Tips for posting to public mailing lists
Re: Spurious warnings
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Spurious warnings


  • Subject: Re: Spurious warnings
  • From: "Jan E. Schotsman" <email@hidden>
  • Date: Tue, 22 Jul 2008 11:09:26 +0200


On Jul 22, 2008, at 10:41 AM, Matt Gough wrote:

But as the original subject of this thread says, this is surely a spurious warning, as he is passing floats into a function that takes floats and only uses the parameters as floats in the implementation.

I could understand it if he was doing this with his parameters:

drawRect = CGRectMake(	0.0,	// Double -> float
			1,	// integer -> float
			shapeBounds.size.width * 2.0f,	// Double temporary -> float
			shapeBounds.size.height + 0.5	// Ditto
			);

I haven't checked, but maybe the ABI wants floating point params to always be doubles and this warning is really telling you this?

In case of a double temporary I still think the warning is incorrect as I have chosen to work in 32-bit precision.
IMHO compiler warnings are not meant to say "why don't you use doubles only as the compiler converts your floats to doubles anyway"?


Jan E.

_______________________________________________
Do not post admin requests to the list. They will be ignored.
Xcode-users mailing list      (email@hidden)
Help/Unsubscribe/Update your Subscription:
This email sent to email@hidden


  • Follow-Ups:
    • Re: Spurious warnings
      • From: "Clark Cox" <email@hidden>
References: 
 >Re: Spurious warnings (From: "Jan E. Schotsman" <email@hidden>)
 >Re: Spurious warnings (From: Jean-Daniel Dupas <email@hidden>)
 >Re: Spurious warnings (From: Matt Gough <email@hidden>)

  • Prev by Date: Re: Spurious warnings
  • Next by Date: Re: Spurious warnings
  • Previous by thread: Re: Spurious warnings
  • Next by thread: Re: Spurious warnings
  • Index(es):
    • Date
    • Thread