aboutsummaryrefslogtreecommitdiff
path: root/core
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | | | | | | | | Merge pull request #18428 from groud/fix_inputRémi Verschelde2018-05-013-5/+7
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixing input strength and the impossibility to erase action events
| * | | | | | | | | | | | | | | | Fixing input strength and the impossibility to erase action eventsgroud2018-04-253-5/+7
| | |_|_|_|_|/ / / / / / / / / / | |/| | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | Merge pull request #18355 from ShyRed/fix18344Rémi Verschelde2018-05-011-9/+9
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Keep alpha when changing h, s or v Color property in GDScript
| * | | | | | | | | | | | | | | | Keep alpha when changing h, s or v on colorShyRed2018-04-221-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When modifying the h, s or v properties of a color in GDScript the alpha value needs to stay unchanged.
* | | | | | | | | | | | | | | | | Merge pull request #18321 from Crazy-P/Fixes-logically-dead-codeRémi Verschelde2018-05-012-2/+6
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / / / / / / |/| | | | | | | | | | | | | | | | Fixes logically dead code (Coverity)
| * | | | | | | | | | | | | | | | Fixes logically dead code (Coverity)Crazy-P2018-04-212-2/+6
| |/ / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | | | | | | | | | | | | | | Merge pull request #18284 from mhilbrunner/pck-versionMax Hilbrunner2018-04-301-4/+4
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / / / / / |/| | | | | | | | | | | | | | | Fix .pck exporting via PCKPacker
| * | | | | | | | | | | | | | | PCKPacker:pck_start(): Update versionMax Hilbrunner2018-04-191-4/+4
| | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | Add option to renormalize mipmaps when generating them for normalmaps.Juan Linietsky2018-04-292-13/+37
| |_|/ / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reduces some aliasing.
* | | | | | | | | | | | | | | Change ".." punctuation for "..." in editor strings (#16507)Hugo Locurcio2018-04-221-1/+1
| | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | Merge pull request #18176 from nikibobi/string-trimGeorge Marques2018-04-223-0/+64
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / / / |/| | | | | | | | | | | | | | Add string trim_prefix, trim_suffix, lstrip and rstrip methods
| * | | | | | | | | | | | | | add string trim_prefix trim_suffix lstrip and rstrip methodsbosak2018-04-173-0/+64
| | |_|_|/ / / / / / / / / / | |/| | | | | | | | | | | |
* | | | | | | | | | | | | | Merge pull request #18280 from groud/fix_inputRémi Verschelde2018-04-191-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / / / |/| | | | | | | | | | | | | Fixes left/up axis not mappable as actions
| * | | | | | | | | | | | | Fixes left/up axis not mappable as actionsGilles Roudiere2018-04-181-2/+2
| | |/ / / / / / / / / / / | |/| | | | | | | | | | |
* / | | | | | | | | | | | Fix some Coverity warnings in String APIRémi Verschelde2018-04-183-29/+13
|/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - StringName::StringName(const StringName &p_name) Non-static class member _data is not initialized in this constructor nor in any functions that it calls. - StringName::_Data() Non-static class member idx is not initialized in this constructor nor in any functions that it calls. - String::num_uint64(...) This less-than-zero comparison of an unsigned value is never true. n % base < 0UL. - String::hex_to_int(...) and String::hex_to_int64(...) Execution cannot reach this statement (deadcode)
* | | | | | | | | | | | Add a macro to deprecate methodsJuan Linietsky2018-04-181-0/+10
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #18204 from tagcup/quat_scaleRémi Verschelde2018-04-183-5/+77
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid converting Quat to Euler angles when not necessary.
| * | | | | | | | | | | | Avoid converting Quat to Euler angles when not necessary.tagcup2018-04-143-5/+77
| | |_|/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also ensure that get_scale doesn't arbitrarlity change the signs of scales, ensuring that the combination of get_rotation and get_scale gives the correct basis. Added various missing functions and constructors. Should close #17968.
* | | | | | | | | | | | Merge pull request #18227 from bojidar-bg/fix-date-unix-crashRémi Verschelde2018-04-181-7/+7
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Fix crash resulting from bad month check in core_bind.cpp
| * | | | | | | | | | | | Fix crash resulting from bad month check in core_bind.cppBojidar Marinov2018-04-171-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also, make it clear that day is 0-based. This might cause very slight differcies in existing games. Fixes #18221
* | | | | | | | | | | | | Merge pull request #16902 from groud/analog_action_systemJuan Linietsky2018-04-188-52/+154
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow actions to provide an analog value
| * | | | | | | | | | | | | Allow actions to provide an analog valueGilles Roudiere2018-04-168-52/+154
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Fix names of Variant operatorsGeorge Marques2018-04-161-1/+2
| |_|/ / / / / / / / / / / |/| | | | | | | | | | | |
* | | | | | | | | | | | | Merge pull request #18170 from Chaosus/improvewrapfGilles Roudiere2018-04-141-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Small performance fix to wrapf
| * | | | | | | | | | | | | Small performance fix to wrapfChaosus2018-04-131-2/+2
| | |_|/ / / / / / / / / / | |/| | | | | | | | | | |
* / | | | | | | | | | | | Revert "Unify http- and percent- encode/decode"Pedro J. Estébanez2018-04-123-7/+72
|/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit b76ee30917c63211ac9e94a21bebbddf518d169f.
* | | | | | | | | | | | iPhone X support and iOS-related fixesRuslan Mustakov2018-04-113-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Starting from April 2018 Apple no longer accepts apps that do not support iPhone X. For games this mainly means respecting the safe area, unobstructed by notch and virtual home button. UI controls must be placed within the safe area so that users can interact with them. This commit: - Adds OS::get_window_safe_area method that returns unobscured area of the window, where interactive controls should be rendered. - Reorganizes how launch screens are exported - the previous way was incorrect and modern iPhones did not pick up the correct screens and because of that used a non-native resolution to render the game. - Adds launch screen options for iPhone X. - Makes launch screens optional in the export template. If not specified, a white screen will be used. - Adds App Store icon (1024x1024) export option as it now has to be bundled with the app instead of being provided in iTunes Connect. - Fixes crash when launching games in iOS Simulator. It happened because controllerWasConnected callback came before the engine was initialized. Now in such case the controllers will be queued up and registered after initialization is done. - Fixes issue with the virtual keyboard where for some reason autocorrection panel would intersect with the keyboard itself and not allow you to use the top row of the keyboard. This is fixed by disabling autocorrection altogether. Closes #17358. Fixes #17428. Fixes #17331.
* | | | | | | | | | | | Update classref and docs, fix missing parameters' namePoommetee Ketson2018-04-111-2/+2
| | | | | | | | | | | |
* | | | | | | | | | | | Implement Input.set_default_cursor_shape to change the default shapeGuilherme Felipe2018-04-102-0/+3
| |_|_|_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #18043
* | | | | | | | | | | Resolves Inccorect Quaternion ConversionCrazy-P2018-04-091-1/+1
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes https://github.com/godotengine/godot/issues/18025
* | | | | | | | | | Merge pull request #16495 from Chaosus/is_point_in_triangle_fixJuan Linietsky2018-04-081-7/+6
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Fix is_point_in_triangle function
| * | | | | | | | | | Fix is_point_in_triangleChaosus2018-02-081-7/+6
| | |/ / / / / / / / | |/| | | | | | | |
* | | | | | | | | | Merge pull request #16898 from Lerc/masterJuan Linietsky2018-04-082-0/+43
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | add Image::bumpmap_to_normalmap conversion function
| * | | | | | | | | | change bumpmap_to_normalmap range from [1...255] to [0...255]Neil Graham2018-03-161-3/+3
| | | | | | | | | | |
| * | | | | | | | | | change to clang formatNeil Graham2018-02-242-20/+20
| | | | | | | | | | |
| * | | | | | | | | | add Image::bumpmap_to_normalmap conversion functionNeil Graham2018-02-212-0/+43
| | | | | | | | | | |
* | | | | | | | | | | Merge pull request #17227 from Faless/multiplayer_apiJuan Linietsky2018-04-083-0/+811
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | [RFC] MultiplayerAPI refactor
| * | | | | | | | | | | Add new MultiplayerAPI classFabio Alessandrelli2018-03-033-0/+811
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Handles all the high level networking stuff
* | | | | | | | | | | | Merge pull request #17232 from playster000/masterJuan Linietsky2018-04-081-49/+17
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | enhance mipmaps, non power of 2 textures
| * | | | | | | | | | | | enhance mipmapsMavropoulos Ioannis2018-03-031-49/+17
| |/ / / / / / / / / / /
* | | | | | | | | | | | Merge pull request #17382 from bojidar-bg/13971-path-array-unsavedJuan Linietsky2018-04-088-13/+27
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Duplicate Arrays and Dictionaries when instancing scene in editor
| * | | | | | | | | | | | Duplicate Arrays and Dictionaries when instancing scene in editorBojidar Marinov2018-03-138-13/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also, add deep (=false) parameter to Array.duplicate and Dictionary.duplicate Fixes #13971
* | | | | | | | | | | | | Merge pull request #17967 from Nibodhika/all-device-modeJuan Linietsky2018-04-082-4/+11
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allows to map an action to all devices.
| * | | | | | | | | | | | | Allows to map an action to all devices.Nibodhika2018-04-072-4/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is accomplished by setting a special value (-1) to the device variable in the InputEvent that's being used to compare with the one received from the OS. This special value is invalid for a regular input, so it should be safe. Implements #17942
* | | | | | | | | | | | | | Merge pull request #17583 from RandomShaper/enhance-uri-utilsJuan Linietsky2018-04-073-74/+29
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enhance uri utils
| * | | | | | | | | | | | | | Fix buggy percent-encodingPedro J. Estébanez2018-04-051-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #17875.
| * | | | | | | | | | | | | | Enhance HTTPClient.query_string_from_dict()Pedro J. Estébanez2018-03-271-1/+21
| | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Unify http- and percent- encode/decodePedro J. Estébanez2018-03-273-70/+5
| | |_|_|_|_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was a percent-prefixed version, which was exposed, and a http-prefixed version which was not (only to GDNative). This commit keeps the percent-prefixed versions, but with the http-prefixed implementations.
* | | | | | | | | | | | | | Merge pull request #17636 from rainerdeyke/fix_issue_17585Juan Linietsky2018-04-071-3/+5
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes moving files causes scene corruption
| * | | | | | | | | | | | | | Fixes scene corruption when updating dependencies.Rainer Deyke2018-03-241-3/+5
| | |_|_|_|_|_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit fixes issue #17585: renaming/moving resources can corrupt scene files. The corruption was caused by 'ResourceFormatLoaderBinary::rename_dependencies' updating the file format version field of the affected scene file to the latest version without actually updating the content of the file to that version, resulting in a file whose content does not match its file format version field. The fix preserves the file format version field and the engine version fields when renaming dependencies.