summaryrefslogtreecommitdiffstats
path: root/modules
Commit message (Collapse)AuthorAgeFilesLines
* Do not make a function that returns Variant::NIL a void function. Fix #22791.J08nY2018-10-111-1/+7
|
* Merge pull request #22865 from neikeq/issue-22854Ignacio Etcheverry2018-10-081-2/+4
|\ | | | | Check if directory exists before trying to delete it
| * Check if directory exists before trying to delete itIgnacio Etcheverry2018-10-081-2/+4
| |
* | Fix multi-statement macro call not being covered.J08nY2018-10-082-4/+6
|/
* Merge pull request #22808 from KellyThomas/vector-oneRémi Verschelde2018-10-072-22/+30
|\ | | | | Add ONE constants to Vector2 and Vector3
| * Update class documentation xmlKelly Thomas2018-10-072-22/+30
| |
* | Merge pull request #22771 from guilhermefelipecgs/fix_inspector_previewRémi Verschelde2018-10-072-2/+2
|\ \ | | | | | | Fix inspector preview
| * | Fix inspector previewGuilherme Felipe2018-10-072-2/+2
| | | | | | | | | | | | Remove script preview to inspector dock
* | | Fix LWSClient connect_to_host string termination.Fabio Alessandrelli2018-10-071-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | Coming from strncpy might get you a non-NULL terminated buffer. The solution, if you accept trunction, is to give one less byte to strncpy and manually set the last char in the buffer to '\0'. If the source string is shorter, than the buffer is padded with '\0' automatically.
* | | Merge pull request #22759 from AndreaCatania/pyfixRémi Verschelde2018-10-0714-191/+180
|\ \ \ | | | | | | | | Some physics fixes for 3.1
| * | | Optimitzed area overlap checksAndrea Catania2018-10-0710-43/+62
| | | |
| * | | Improved code that handles collision shapes, fixes #21945Andrea Catania2018-10-0610-144/+109
| | | |
| * | | Improved trimesh stabilityAndrea Catania2018-10-051-4/+10
| | | |
| * | | Removed wrong equal vertex thresold to GJK solverAndrea Catania2018-10-051-1/+0
| | | |
| * | | Added correct allowed penetration parameter to cast motion APIAndrea Catania2018-10-051-1/+1
| | | |
* | | | C# bindings generator fixesIgnacio Etcheverry2018-10-071-4/+4
| | | | | | | | | | | | | | | | | | | | - Fix unused bool local for MonoBoolean argument. - Append U to API hashes. Fixes warning: 'integer constant is so large that it is unsigned'
* | | | Merge pull request #22752 from aaronfranke/equals-redundantRémi Verschelde2018-10-077-15/+15
|\ \ \ \ | | | | | | | | | | Remove redundant "== true" and "== false" code
| * | | | Remove redundant "== false" codeAaron Franke2018-10-066-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some of this code has been re-organized. f
| * | | | Remove redundant "== true" codeAaron Franke2018-10-064-7/+7
| | | | | | | | | | | | | | | If it can be compared to a boolean, it can be evaluated as one in-place.
* | | | | Merge pull request #22812 from neikeq/issue-22781Rémi Verschelde2018-10-074-81/+17
|\ \ \ \ \ | |/ / / / |/| | | | Mono: Fix crash on NodePath/RID disposal during Godot shutdown
| * | | | Mono: Fix crash on NodePath/RID disposal during Godot shutdownIgnacio Etcheverry2018-10-064-81/+17
| | |_|/ | |/| |
* | | | Fix compiler warnings in GDScript moduleGeorge Marques2018-10-061-10/+19
| | | |
* | | | Merge pull request #22796 from ↵Rémi Verschelde2018-10-061-5/+5
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | KoderaSoftwareUnlimited/fix-segfault-mouse-move-related Reverts warning fix on release builds that caused segfault in release build
| * | | | Revert cause of #22794Mariusz Chwalba2018-10-061-5/+5
| | | | |
* | | | | Merge pull request #22773 from KellyThomas/docs-linksRémi Verschelde2018-10-061-1/+1
|\ \ \ \ \ | |/ / / / |/| | | | [Docs] Fix some broken links
| * | | | [Docs] Fix some broken linksKelly Thomas2018-10-061-1/+1
| | |/ / | |/| |
* | | | Fixes to baker, restored xatlas and fixed bake options.Juan Linietsky2018-10-053-7/+18
| |/ / |/| |
* | | Don't delete FileAccess when it is nullMarc Gilleron2018-10-052-2/+0
| |/ |/|
* | Merge pull request #22769 from neikeq/issue-22765Rémi Verschelde2018-10-053-7/+7
|\ \ | | | | | | Fix C# API assembly build errors in generics
| * | Fix C# API assembly build errors in genericsIgnacio Etcheverry2018-10-053-7/+7
| | |
* | | Fix build error for windows mono export templatesIgnacio Etcheverry2018-10-051-1/+1
|/ /
* | Merge pull request #22760 from zochris/loosen-generic-guardRémi Verschelde2018-10-052-9/+9
|\ \ | | | | | | Loosen generic guard on some methods
| * | loosen generic guard on method for ResourceLoaderzochris2018-10-051-1/+1
| | | | | | | | | For more details see #22658
| * | loosen generic guard on Node extension methodszochris2018-10-051-8/+8
| |/ | | | | For more details see #22658
* | Merge pull request #22688 from neikeq/lotsofgoodiesRémi Verschelde2018-10-0515-242/+654
|\ \ | |/ |/| Mono: Editor and export template dependencies and fixes
| * Mono: Editor and export template dependencies and fixesIgnacio Etcheverry2018-10-0315-242/+654
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Bundle editor dependencies: - 'GodotSharp': Root data directory for the editor - 'Tools': Editor dependencies. Only GodotSharp.dll for now. - 'Api': Prebuilt GodotSharp and GodotSharpEditor API assemblies. - 'Mono': Mono files to bundle with the editor. - 'bin': (Optional, not used for now) Mono bin directory. - 'etc': Mono configuration files. - 'lib': Mono dependency shared libraries. - 'lib/mono/4.5': Framework assemblies. - Added build option to copy the required files from the mono installation to 'GodotSharp/Mono'. Enable with 'copy_mono_root=yes'. Disabled by default. - Export template dependencies: - 'data_AppName'/'data_Godot': - 'Mono': Mono files to bundle with the game. - 'etc': Mono configuration files. - 'lib': Mono dependency shared libraries. - The data directory is generated when compiling and must be bundled with the export templates. In the case of OSX, the data directory must be placed inside the 'osx.zip' export template. - In OSX, alternative location for directories (needed for app bundles) are: - 'data_AppName/Mono/etc' --> '../Resources/GodotSharp/Mono/etc' - 'data_AppName/Mono/lib' --> '../Frameworks/GodotSharp/Mono/lib' - The editor can bundle prebuilt API assemblies. - Generate them with a tools build by running: `--generate-cs-core-api <GodotSharp_OutputDir> --generate-cs-editor-api <GodotSharpEditor_OutputDir> <GodotSharp_OutputDir>/bin/Release/GodotSharp.dll` (This command will be simplified in the future and both projects will be in the same solution) - Build the solutions and copy the output files to '#bin/GodotSharp/Api'. - Fixed API assembly being added twice during the export process.
* | Fix some cppcheck errorsSimon Wenner2018-10-041-0/+1
| | | | | | | | | | | | cppcheck: fix broken pre-processor statement cppcheck: fix Uninitialized struct members in canvas editor plugin cppcheck: fix uninitialized struct member: method_api.is_virtual
* | Fix GCC 8 warnings about potentially unitialized variablesRémi Verschelde2018-10-042-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes the following GCC 8 warnings: ``` core/image.cpp:730:44: warning: 'mip1_weight' may be used uninitialized in this function [-Wmaybe-uninitialized] core/image.cpp:293:20: warning: 'mip2' may be used uninitialized in this function [-Wmaybe-uninitialized] core/image.cpp:293:20: warning: 'mip1' may be used uninitialized in this function [-Wmaybe-uninitialized] editor/audio_stream_preview.cpp:58:19: warning: 'vmax' may be used uninitialized in this function [-Wmaybe-uninitialized] editor/audio_stream_preview.cpp:85:19: warning: 'vmin' may be used uninitialized in this function [-Wmaybe-uninitialized] editor/editor_themes.cpp:306:53: warning: 'preset_contrast' may be used uninitialized in this function [-Wmaybe-uninitialized] editor/plugins/animation_blend_space_2d_editor.cpp:459:27: warning: 'prev_idx' may be used uninitialized in this function [-Wmaybe-uninitialized] editor/plugins/animation_blend_space_2d_editor.cpp:443:27: warning: 'prev_idx' may be used uninitialized in this function [-Wmaybe-uninitialized] main/tests/test_oa_hash_map.cpp:57:29: warning: 'value' may be used uninitialized in this function [-Wmaybe-uninitialized] modules/csg/csg.cpp:764:40: warning: 'max_angle' may be used uninitialized in this function [-Wmaybe-uninitialized] modules/csg/csg_shape.cpp:1945:3: warning: 'face_count' may be used uninitialized in this function [-Wmaybe-uninitialized] scene/3d/voxel_light_baker.cpp:1593:8: warning: 'cone_aperture' may be used uninitialized in this function [-Wmaybe-uninitialized] scene/3d/voxel_light_baker.cpp:1592:6: warning: 'cone_dir_count' may be used uninitialized in this function [-Wmaybe-uninitialized] scene/animation/animation_blend_space_2d.cpp:471:8: warning: 'mind' may be used uninitialized in this function [-Wmaybe-uninitialized] core/os/memory.cpp:94: warning: ignoring #pragma clang diagnostic [-Wunknown-pragmas] core/os/memory.cpp:95: warning: ignoring #pragma clang diagnostic [-Wunknown-pragmas] core/os/memory.cpp:98: warning: ignoring #pragma clang diagnostic [-Wunknown-pragmas] ```
* | Fix some OSX and iOS Clang warningsRémi Verschelde2018-10-041-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes the following XCode 9.4.1 warnings: ``` core/os/memory.cpp:175:13: warning: unused variable 's' [-Wunused-variable] drivers/coremidi/core_midi.cpp:68:14: warning: comparison between NULL and non-pointer ('MIDIEndpointRef' (aka 'unsigned int') and NULL) [-Wnull-arithmetic] drivers/gles2/rasterizer_gles2.cpp:77:24: warning: unused function '_gl_debug_print' [-Wunused-function,34] drivers/unix/thread_posix.cpp:106:12: warning: unused variable 'running_thread' [-Wunused-variable,34] modules/gdnative/nativescript/nativescript.h:371:16: warning: in-class initialization of non-static data member is a C++11 extension [-Wc++11-extensions] platform/iphone/gl_view.mm:56:14: warning: unused variable 'video_previous_volume' [-Wunused-variable,34] platform/iphone/gl_view.mm:251:12: warning: unused function 'get_first_id' [-Wunused-function,34] platform/iphone/main.m:45:15: warning: unused variable 'app' [-Wunused-variable,34] platform/osx/os_osx.mm:79:15: warning: unused function 'convertRectToBacking' [-Wunused-function] ```
* | Fix warnings about non-static data member initializers in nativescriptRémi Verschelde2018-10-042-11/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | Fixes the following GCC/Clang warnings: ``` modules/gdnative/nativescript/nativescript.h:280:37: warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11 modules/gdnative/nativescript/nativescript.h:281:37: warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11 modules/gdnative/nativescript/nativescript.h:283:42: warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11 modules/gdnative/nativescript/nativescript.h:285:38: warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11 modules/gdnative/nativescript/nativescript.h:287:38: warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11 modules/gdnative/nativescript/nativescript.h:290:45: warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11 modules/gdnative/nativescript/nativescript.h:291:44: warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11 ```
* | Merge pull request #22681 from akien-mga/fix-warningsRémi Verschelde2018-10-033-20/+17
|\ \ | | | | | | Fix warnings on release builds and some MSVC warnings
| * | Fix warnings on release builds (not DEBUG_ENABLED)Rémi Verschelde2018-10-033-20/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes the following Clang 5 warnings: ``` modules/bmp/image_loader_bmp.cpp:46:60: warning: comparison of unsigned expression < 0 is always false [-Wtautological-compare] modules/bmp/image_loader_bmp.cpp:48:61: warning: comparison of unsigned expression < 0 is always false [-Wtautological-compare] drivers/png/image_loader_png.cpp:231:20: warning: comparison of unsigned expression >= 0 is always true [-Wtautological-compare] scene/gui/graph_edit.cpp:1045:8: warning: comparison of constant 0 with expression of type 'bool' is always false [-Wtautological-constant-out-of-range-compare] core/class_db.cpp:812:13: warning: unused variable 'check' [-Wunused-variable] core/io/file_access_pack.cpp:172:11: warning: unused variable 'ver_rev' [-Wunused-variable] core/math/bsp_tree.cpp:195:13: warning: unused variable 'plane' [-Wunused-variable] core/math/bsp_tree.cpp:168:6: warning: unused variable 'plane_count' [-Wunused-variable] modules/gdscript/gdscript_function.cpp:685:10: warning: unused variable 'ok' [-Wunused-variable] modules/gdscript/gdscript_function.cpp:706:10: warning: unused variable 'ok' [-Wunused-variable] modules/gdscript/gdscript_function.cpp:755:19: warning: unused variable 'var_type' [-Wunused-variable] modules/gdscript/gdscript_function.cpp:1306:12: warning: unused variable 'err' [-Wunused-variable] modules/gdscript/gdscript_function.cpp:158:15: warning: unused function '_get_var_type' [-Wunused-function] modules/gdscript/gdscript_parser.cpp:750:20: warning: unused variable 'lv' [-Wunused-variable] modules/gdscript/gdscript_parser.cpp:59:15: warning: unused function '_find_function_name' [-Wunused-function] scene/main/node.cpp:2489:13: warning: unused function '_Node_debug_sn' [-Wunused-function] ```
* | | Fixed DDS loader according to issue #22530qonnop2018-10-031-1/+1
|/ / | | | | | | | | | | width and height attributes have to be switched according to the official DDS header docs: https://docs.microsoft.com/en-us/windows/desktop/direct3ddds/dds-header
* | Fix warnings in Android platformRémi Verschelde2018-10-031-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes the following Clang 7 warnings: ``` platform/android/os_android.h:240:16: warning: 'OS_Android::native_video_play' hides overloaded virtual function [-Woverloaded-virtual] platform/android/os_android.h:241:15: warning: 'OS_Android::native_video_is_playing' hides overloaded virtual function [-Woverloaded-virtual] platform/android/audio_driver_opensl.cpp:104:3: warning: suggest braces around initialization of subobject [-Wmissing-braces] platform/android/audio_driver_opensl.cpp:129:10: warning: unused variable 'numOutputs' [-Wunused-variable] platform/android/audio_driver_opensl.cpp:130:11: warning: unused variable 'deviceID' [-Wunused-variable] platform/android/java_glue.cpp:795:10: warning: unused variable 'clsio' [-Wunused-variable] platform/android/java_glue.cpp:890:12: warning: unused variable 'gob' [-Wunused-variable] platform/android/java_glue.cpp:592:13: warning: unused variable 'resized' [-Wunused-variable] platform/android/java_glue.cpp:593:13: warning: unused variable 'resized_reload' [-Wunused-variable] modules/mobile_vr/mobile_vr_interface.cpp:401:8: warning: unused variable 'aspect_ratio' [-Wunused-variable] drivers/unix/dir_access_unix.cpp:394:2: warning: THIS IS BROKEN [-W#warnings] ```
* | Merge pull request #22672 from akien-mga/fix-warningsRémi Verschelde2018-10-032-3/+3
|\ \ | | | | | | Fix various warnings reported by CI in #22620
| * | Fix GDNative build warning on Android [-Wignored-attributes]Rémi Verschelde2018-10-031-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes the following kind of warning spam: ``` modules/gdnative/include/gdnative/color.h:61:6: warning: calling convention 'sysv_abi' ignored for this target [-Wignored-attributes] void GDAPI godot_color_new_rgba(godot_color *r_dest, const godot_real p_r, const godot_real p_g, const godot_real p_b, const godot_real p_a); ^ modules/gdnative/include/gdnative/gdnative.h:52:15: note: expanded from macro 'GDAPI' ^ modules/gdnative/include/gdnative/gdnative.h:51:38: note: expanded from macro 'GDCALLINGCONV' ^ ```
| * | Fix some more warnings reported by CIRémi Verschelde2018-10-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes the following Clang 7 warnings: ``` core/io/marshalls.cpp:872:10: warning: unused variable 'f' [-Wunused-variable] core/ustring.cpp:1831:2: warning: 'register' storage class specifier is deprecated and incompatible with C++17 [-Wdeprecated-register] core/ustring.cpp:1832:2: warning: 'register' storage class specifier is deprecated and incompatible with C++17 [-Wdeprecated-register] drivers/gles3/rasterizer_gles3.cpp:82:24: warning: unused function '_gl_debug_print' [-Wunused-function,34] main/main.cpp:118:13: warning: unused variable 'auto_build_solutions' [-Wunused-variable] modules/csg/csg_gizmos.cpp:225:46: warning: 'current' may be used uninitialized in this function [-Wmaybe-uninitialized] ```
* | | Merge pull request #22666 from JFonS/noisetexture_remove_sizeRémi Verschelde2018-10-034-19/+8
|\ \ \ | | | | | | | | Remove redundant size property from NoiseTexture
| * | | Remove redundant size property from NoiseTextureJFonS2018-10-034-19/+8
| |/ /
* | | Merge pull request #22653 from neikeq/mono-mod-fix-gcc-warningsRémi Verschelde2018-10-034-13/+19
|\ \ \ | |/ / |/| | Fix GCC compiler warning in mono module