Jump to content

Framwerk compile warnings


Roland
 Share

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

SetShaderFloat(postfilter,"saturation",(flt)saturation);
SetShaderFloat(postfilter,"brightness",(flt)brightness);
SetShaderFloat(postfilter,"contrast",(flt)contrast);

 

Renderer.cpp 1085

SetColor(Vec4(1,1,1,(flt)irisadjustmentspeed*AppSpeed()));

  • Upvote 1

Roland Strålberg
Website: https://rstralberg.com

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.

Guest
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.

 Share

×
×
  • Create New...