Jump to content

Framwerk compile warnings


Recommended Posts

To get rid of this type of warnings


cpp\renderer.cpp(457) : warning C4244: 'argument' : conversion from 'double' to 'flt', possible loss of data


following lines needs to be changed in Renderer.cpp


Renderer.cpp 457

ScaleEntity( skybox, Vec3(farrange/2.0f) );


Renderer.cpp 798-800



Renderer.cpp 1085


  • Upvote 1

AV MX Linux

Link to comment
Share on other sites

I don't know why you get warning messages, because I don't get any.



Mayby your are 'cheating' by having this line at top..

#pragma warning(disable:4244)


or maybe some compiler setting


Edited by Roland Strålberg

AV MX Linux

Link to comment
Share on other sites

I don't disable type conversion warnings, but make the code so that it has no type conversions.



Isn't that what Roland's post was giving instructions for? :D

AMD Bulldozer FX-4 Quad Core 4100 Black Edition

2 x 4GB DDR3 1333Mhz Memory

Gigabyte GeForce GTX 550 Ti OC 1024MB GDDR5

Windows 7 Home 64 bit


BlitzMax 1.50 • Lua 5.1 MaxGUI 1.41 • UU3D Pro • MessiahStudio Pro • Silo Pro

3D Coat • ShaderMap Pro • Hexagon 2 • Photoshop, Gimp & Paint.NET


LE 2.5/3.4 • Skyline UE4 • CE3 SDK • Unity 5 • Esenthel Engine 2.0


Marleys Ghost's YouTube Channel Marleys Ghost's Blog


"I used to be alive like you .... then I took an arrow to the head"

Link to comment
Share on other sites

Ok, I found the reason why Roland gets those warnings. He is using Framewerk directly, and not via gamelib. When I tried to make a new project which had the Framewerk files directly included as source, then I got those warnings too.


I'll fix those warnings and send Josh the updated files.

EDIT: Sent new renderer.cpp to Josh

Ryzen 9 RX 6800M ■ 16GB XF8 Windows 11 ■
Ultra ■ LE 2.53DWS 5.6  Reaper ■ C/C++ C# ■ Fortran 2008 ■ Story ■
■ Homepage: https://canardia.com ■

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.


  • Create New...