summaryrefslogtreecommitdiff
path: root/servers/visual
AgeCommit message (Collapse)Author
2018-05-01Merge pull request #18291 from akien-mga/coverity-uninitialized-scalar-varRémi Verschelde
Fix Coverity reports of uninitialized scalar variable
2018-05-01Merge pull request #18321 from Crazy-P/Fixes-logically-dead-codeRémi Verschelde
Fixes logically dead code (Coverity)
2018-04-22Change ".." punctuation for "..." in editor strings (#16507)Hugo Locurcio
2018-04-21Fixes logically dead code (Coverity)Crazy-P
Fixes reported logically dead codes by Coverity * image.cpp: Doesn't really need any modification. But to remove the bug report then we have to move the MAX call away from the for loop statement. * rasterizer_gles3.cpp: Removes unnecessary elif condition since it is checked earlier in the function * collada.cpp: If stamement never reached due to macro ERR_CONTINUE does the same. * navigation_mesh.cpp: Variables should always be null - however, also checked for the very same condition in their function call. Leaving this for review (whether the function call is necessary or not) * path_editor_plugin.cpp: If cancel is true, then it should restore the edited value to the original provided. http://docs.godotengine.org/en/3.0/classes/class_editorspatialgizmo.html#class-editorspatialgizmo-commit-handle * spatial_editor_gizmos.cpp: the very condition of i >= 3 is predetermined in the if case right before it. Thus case 1 is always '1' and case 2 is always '-1' * grid_map_editor.cpp: Same as above in spatial_editor_gizmos.cpp * voxel_light_baker.cpp: Same as above in spatial_editor_gizmos.cpp * visual_server.cpp: Same as above in spatial_editor_gizmos.cpp * visual_script_expression.cpp: char '-' is already true in the switch case mechanism. Thus it can never reach to default case. * particles.cpp: Case 'PARAM_MAX' is unreachable due to index checking right before the switch execution. * shader_language.cpp: Invalid index is handled in switch default case. `type < TYPE_FLOAT && type > TYPE_VEC4` -> `(type < TYPE_FLOAT || type > TYPE_VEC4`) Fixes the "always false problem" in TODO comment.
2018-04-19Fix Coverity reports of uninitialized scalar variableRémi Verschelde
Fixes most current reports on Coverity Scan of uninitialized scalar variable (CWE-457): https://cwe.mitre.org/data/definitions/457.html These happen most of the time (in our code) when instanciating structs without a constructor (or with an incomplete one), and later returning the instance. This is sometimes intended though, as some parameters are only used in some situations and should not be double-initialized for performance reasons (e.g. `constant` in ShaderLanguage::Token).
2018-04-18Merge pull request #17391 from PJB3005/18-03-09-fix-canvas-light-shadersRémi Verschelde
Fixes canvas light shaders.
2018-04-17Fix typos in shader_language.cppPoommetee Ketson
2018-04-13Fixes canvas light shaders.Pieter-Jan Briers
Fixes #16904 Restore more out functionality, fix built-ins. Requested changes, I think?
2018-04-08Merge pull request #17421 from Chaosus/fixshaderbugsJuan Linietsky
Fix few bugs in shader definitions
2018-03-28Removed redundant abs shader builtinChaosus
2018-03-23Merge pull request #17532 from BastiaanOlij/arvr_enhancementsRémi Verschelde
A few enhancements on the ARVR Server
2018-03-19Fix crash in `canvas_item_add_polyline` when passing more points than colorsrobfram
When `p_points.size() > p_colors.size()`, it crashed with invalid array access to `p_colors`. Also, when `p_colors` was an empty `Vector` it crashed due a missing `else` checking the `size` condition, as the code handling that special case exists. This PR fixes the missing `else` for `p_colors.size == 0` and, following the `canvas_item_add_multiline` spirit, it only uses the first color for the whole polyline if points and colors differ in size. Fix #17621.
2018-03-15Merge pull request #17248 from Cygon/implement-extra-culling-marginHein-Pieter van Braam
Apply culling margin (fixes issue #16115)
2018-03-15Hinted shader uniforms can have a default valueJFonS
2018-03-15A few fixes on the ARVR ServerBastiaan Olij
2018-03-13Merge pull request #17402 from Chaosus/fixmixRémi Verschelde
Fix invalid mix function overload
2018-03-13Fixed several leaks in VisualServerScene, RasterizerSceneGLES3 and ↵Wilson E. Alvarez
RasterizerGLES3
2018-03-10Fix few bugs in shader definitionsChaosus
2018-03-10Fix invalid mix function overloadChaosus
2018-03-07Make the shader token names consistent.Saracen
2018-03-04Implement VisualServerScene::instance_set_extra_visibility_margin() method ↵Markus Ewald
to actually apply extra culling margin and mark instance AABB dirty
2018-03-03Fix floatBitsToUint functionChaosus
2018-02-212D Skeletons WORK IN PROGRESSJuan Linietsky
2018-02-21Add base support for 2D meshes in Godot, including Sprite -> Mesh2D conversion.Juan Linietsky
2018-02-20Revert "Handle single-argument constructors in uniform default values"Juan Linietsky
2018-02-20Merge pull request #15985 from Nallebeorn/uniforms-single-argument-constructorsRémi Verschelde
Handle single-argument constructors in uniform default values
2018-02-18Added missing shader functionsChaosus
2018-02-01Fix inconsistencies and typos in argument namesPaolo Perkovic
2018-01-22Handle single-argument constructors in shaders.Benjamin
Expand these (fill vectors, fill matrix diagonals) in _reduce_expression() so they're handled properly even as uniform default values (where they previously caused a crash).
2018-01-18Fix typos in code and docs with codespellRémi Verschelde
Using v1.11.0 from https://github.com/lucasdemarchi/codespell
2018-01-18Fix wrong viewport transparent-bgbinbitten
2018-01-06Properly dispose of instance capture data, fixes #14795Juan Linietsky
2018-01-05Add missing copyright headers and fix formattingRémi Verschelde
Using `misc/scripts/fix_headers.py` on all Godot files. Some missing header guards were added, and the header inclusion order was fixed in the Bullet module.
2018-01-04Merge pull request #15251 from binbitten/fix-assign-vardeclRémi Verschelde
Fix premature declaration of shader variables created with assignment
2018-01-04Merge pull request #15191 from Jerome67000/z_renamingRémi Verschelde
renames "z" Node2D property to "z_index"
2018-01-03Fix broken Info.plist after copyright updateRazah
Fixes #15293. + related misc fixes by @akien-mga.
2018-01-03#15078 renamed "z" -> "z_index" property in Node2DJerome67000
2018-01-03Fix premature declaration of shader variables created with assignmentbinbitten
2018-01-01Update copyright statements to 2018Rémi Verschelde
Happy new year to the wonderful Godot community!
2017-12-29Fixed wrong highlighted line when erroring and using single-line comments, ↵binbitten
fixes #15167
2017-12-27Merge pull request #14971 from mrcdk/particles_emittingNoshyaar
Set particles emitting to false when particles finish emitting with one-shot enabled
2017-12-26Fix sidedness check in material. Also remove SIDE built-in.Juan Linietsky
2017-12-23Set particles emitting to false when particles finish emitting with one-shot ↵MrCdK
enabled
2017-12-17Cleanup some #if 0'd codeRémi Verschelde
2017-12-17Shader Language: Add in-for declared variables within for-block scope.Enzo Nocera
2017-12-16Add epic hack so vsync can be toggled in run-time from script. Fixes #14458.Juan Linietsky
Call needs to be routed via visual server to reach the proper thread.
2017-12-15Made built-in identifiers properly constant in shaders. Fixes #14449, closes ↵Juan Linietsky
#14629
2017-12-14-Add lightmapperJuan Linietsky
-Fixes to unwrapper (remove degenerates), makes Thekla not crash -Added optional cancel button in EditorProgress -Added function to force processing of events (needed for cancel button)
2017-12-09Fix #14429 GIProbe does not work with SpotLightsPrzemyslaw Czarnota
2017-12-09-Ability to and unwrap lightmap coordinates on importJuan Linietsky
-Added unwrap functionality to Mesh -Ability to display and debug mesh UVs -Added multiline draw, so it's easier and faster to draw UVs -Many fixes to SurfaceTool -Fixes to Thekla Unwrap, but it's a piece of ass and it keeps crashing. Will have to go away