godotengine/godot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Added missing FIXMEs in PR #7878

Browse Source
This commit is contained in:
Ferenc Arn 2017-03-02 16:13:01 -06:00
parent 74eace2b14
commit 790611ef26
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
drivers/gles3/rasterizer_scene_gles3.cpp
View File

@ -2806,7 +2806,7 @@ void RasterizerSceneGLES3::_copy_to_front_buffer(Environment *env) {
//no environment, simply convert from linear to srgb //no environment, simply convert from linear to srgb
storage->shaders.copy.set_conditional(CopyShaderGLES3::LINEAR_TO_SRGB,true); storage->shaders.copy.set_conditional(CopyShaderGLES3::LINEAR_TO_SRGB,true);
} else { } else {
/* Why are both statements equal? */ /* FIXME: Why are both statements equal? */
storage->shaders.copy.set_conditional(CopyShaderGLES3::LINEAR_TO_SRGB,true); storage->shaders.copy.set_conditional(CopyShaderGLES3::LINEAR_TO_SRGB,true);
} }

2
modules/visual_script/visual_script_builtin_funcs.cpp
View File

@ -329,7 +329,7 @@ PropertyInfo VisualScriptBuiltinFunc::get_input_value_port_info(int p_idx) const
case LOGIC_CLAMP: { case LOGIC_CLAMP: {
if (p_idx==0) if (p_idx==0)
return PropertyInfo(Variant::REAL,"a"); return PropertyInfo(Variant::REAL,"a");
else if (p_idx==0) // is it ok to test p_idx == 0 twice? else if (p_idx==0) // FIXME: is it ok to test p_idx == 0 twice?
return PropertyInfo(Variant::REAL,"min"); return PropertyInfo(Variant::REAL,"min");
else else
return PropertyInfo(Variant::REAL,"max"); return PropertyInfo(Variant::REAL,"max");