[Development] Changing qreal to a float

[Craig.Scott at csiro.au]

On 20/02/2012, at 1:29 PM, Lincoln Ramsay wrote:

> On 02/17/2012 09:58 PM, ext lars.knoll at nokia.com wrote:
>> On the idea itself I'd like to hear some more opinions. Removing qreal
>> would cause some SC breakage, but you would also get a compile error on
>> places that could break. Unfortunately the signal to noise ratio would
>> most likely be pretty bad.
> 
> I vote for removing it. I think that's better than making it just float 
> or double.

Rather than an outright removal, perhaps it would be better to simply #ifdef it out with some way for devs to re-enable it. Thus, by default it will be as though it was no longer defined (hence prompts people to update their code), but for those who really do have qreal embedded throughout their own code base, there's an easy mechanism for them to keep their code building without hacking into Qt's headers, etc.

Just my $0.02.

--
Dr Craig Scott
Computational Software Engineering Team Leader, CSIRO (CMIS)
Melbourne, Australia