Jump to content

Recommended Posts

Posted

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

Posted (edited)

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

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

Posted

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

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 ■

Posted

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"

Posted

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 ■

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.

×
×
  • Create New...