GlyphArrangement C4244 compiler warnings


#1

Apologies if you're already aware of this, but after grabbing a git update earlier today, I started seeing compiler warnings (VS Express 2013) in relation to this class, as follows:

1>c:\juce\modules\juce_graphics\fonts/juce_GlyphArrangement.cpp(377): warning C4244: 'argument' : conversion from 'const float' to 'int', possible loss of data (..\..\..\..\..\..\..\JUCE\modules\juce_graphics\juce_graphics.cpp)
1>c:\juce\modules\juce_graphics\fonts/juce_GlyphArrangement.cpp(408): warning C4244: 'argument' : conversion from 'const float' to 'int', possible loss of data (..\..\..\..\..\..\..\JUCE\modules\juce_graphics\juce_graphics.cpp)
1>c:\juce\modules\juce_graphics\fonts/juce_GlyphArrangement.cpp(434): warning C4244: 'argument' : conversion from 'int' to 'float', possible loss of data (..\..\..\..\..\..\..\JUCE\modules\juce_graphics\juce_graphics.cpp)
1>c:\juce\modules\juce_graphics\fonts/juce_GlyphArrangement.cpp(635): warning C4244: 'argument' : conversion from 'int' to 'float', possible loss of data (..\..\..\..\..\..\..\JUCE\modules\juce_graphics\juce_graphics.cpp)
1>c:\juce\modules\juce_graphics\fonts/juce_GlyphArrangement.cpp(652): warning C4244: 'initializing' : conversion from 'int' to 'float', possible loss of data (..\..\..\..\..\..\..\JUCE\modules\juce_graphics\juce_graphics.cpp)
1>c:\juce\modules\juce_graphics\fonts/juce_GlyphArrangement.cpp(662): warning C4244: '=' : conversion from 'int' to 'float', possible loss of data (..\..\..\..\..\..\..\JUCE\modules\juce_graphics\juce_graphics.cpp)
1>c:\juce\modules\juce_graphics\fonts/juce_GlyphArrangement.cpp(730): warning C4244: 'argument' : conversion from 'int' to 'float', possible loss of data (..\..\..\..\..\..\..\JUCE\modules\juce_graphics\juce_graphics.cpp)
1>c:\juce\modules\juce_graphics\fonts/juce_GlyphArrangement.cpp(740): warning C4244: 'argument' : conversion from 'int' to 'float', possible loss of data (..\..\..\..\..\..\..\JUCE\modules\juce_graphics\juce_graphics.cpp)

They stem from the check-in from a couple of days ago: https://github.com/julianstorer/JUCE/commit/02460bb95b7b7dfb174b293d9fdbb002f0bbe444


#2

Yes - sorry, had been meaning to tidy that up today, will do so imminently..


#3

Thanks for the update, no compiler warnings now!