Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | | Merge pull request #29780 from GodotExplorer/gdscript-lsp | Rémi Verschelde | 2019-08-28 | 1 | -0/+4 | |
|\ \ \ \ | |/ / / |/| | | | Add Language Server Protocol for GDScript | |||||
| * | | | Add GDScript Language Protocol plugin | geequlim | 2019-08-11 | 1 | -0/+4 | |
| | | | | ||||||
* | | | | Make 'break' and 'continue' be marked as safe | Michael Alexsander Silva Dias | 2019-08-27 | 1 | -2/+4 | |
| | | | | ||||||
* | | | | Improve writing style in GDScript error/warning messages | Hugo Locurcio | 2019-08-23 | 1 | -224/+224 | |
| | | | | | | | | | | | | | | | | This uses double quotes everywhere for consistency. | |||||
* | | | | Replace last occurrences of 'ERR_EXPLAIN' with 'ERR_FAIL_*_MSG' | Rémi Verschelde | 2019-08-17 | 1 | -1/+1 | |
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | The last remaining ERR_EXPLAIN call is in FreeType code and makes sense as is (conditionally defines the error message). There are a few ERR_EXPLAINC calls for C-strings where String is not included which can stay as is to avoid adding additional _MSGC macros just for that. Part of #31244. | |||||
* | | | Merge pull request #31227 from profan/fix/err-explain-usages | Rémi Verschelde | 2019-08-09 | 1 | -10/+4 | |
|\ \ \ | | | | | | | | | Replace 'ERR_EXPLAIN' with 'ERR_FAIL_*_MSG' in "platform", "modules/gdnative", "modules/gdscript" directories. | |||||
| * | | | Replace 'ERR_EXPLAIN' with 'ERR_FAIL_*_MSG' in "platform", ↵ | Robin Hübner | 2019-08-09 | 1 | -10/+4 | |
| | |/ | |/| | | | | | | | "modules/gdnative", "modules/gdscript" directories. | |||||
* / | | Expression before 'is' may be null | Cameron Reikes | 2019-08-09 | 1 | -0/+4 | |
|/ / | ||||||
* | | Merge pull request #31041 from 2shady4u/master | Rémi Verschelde | 2019-08-05 | 1 | -0/+1 | |
|\ \ | | | | | | | Mark class_name line as safe in editor | |||||
| * | | Mark class_name line as safe in editor | shaderbeast | 2019-08-02 | 1 | -0/+1 | |
| | | | ||||||
* | | | Merge pull request #30576 from qarmin/lgtm_coverage | Rémi Verschelde | 2019-07-20 | 1 | -2/+0 | |
|\ \ \ | | | | | | | | | Changed some code reported by LGTM and Coverity | |||||
| * | | | Changed some code showed in LGTM and Coverage | qarmin | 2019-07-20 | 1 | -2/+0 | |
| |/ / | ||||||
* / / | Fix unreachable code detection in match statements | Shiqing | 2019-07-13 | 1 | -2/+19 | |
|/ / | ||||||
* | | Merge pull request #30352 from bojidar-bg/28727-enum-value-check | Rémi Verschelde | 2019-07-06 | 1 | -0/+3 | |
|\ \ | | | | | | | Fix parsing of enums allowing for juxtaposed identifiers | |||||
| * | | Fix parsing of enums allowing for juxtaposed identifiers | Bojidar Marinov | 2019-07-05 | 1 | -0/+3 | |
| | | | | | | | | | | | | Fixes #28727 | |||||
* | | | Merge pull request #30289 from bojidar-bg/29586-class-name-constant | Rémi Verschelde | 2019-07-03 | 1 | -9/+14 | |
|\ \ \ | | | | | | | | | Fix inheriting from class_name messing up constants | |||||
| * | | | Fix inheriting from class_name messing up constants | Bojidar Marinov | 2019-07-03 | 1 | -9/+14 | |
| |/ / | | | | | | | | | | Fixes #29586 | |||||
* / / | Fix parsing of arguments in constant expressions | Bojidar Marinov | 2019-07-03 | 1 | -20/+12 | |
|/ / | | | | | | | Fixes #8006 | |||||
* | | Merge pull request #29815 from NilsIrl/plus_file_1 | Rémi Verschelde | 2019-07-01 | 1 | -1/+1 | |
|\ \ | | | | | | | Replace ` + "/" + ` with `String::file_add()` | |||||
| * | | Replace ` + "/" + ` with `String::file_add()` | Nils ANDRÉ-CHANG | 2019-06-23 | 1 | -1/+1 | |
| | | | ||||||
* | | | Fixed regression bug caused in #30095 and actually fix the issue it was ↵ | unknown | 2019-06-28 | 1 | -19/+7 | |
| | | | | | | | | | | | | supposed to fix(#26850) | |||||
* | | | Merge pull request #29941 from qarmin/redundant_code_and_others | Rémi Verschelde | 2019-06-27 | 1 | -6/+3 | |
|\ \ \ | | | | | | | | | Remove redundant code, possible NULL pointers and others | |||||
| * | | | Some code changed with Clang-Tidy | qarmin | 2019-06-26 | 1 | -6/+3 | |
| |/ / | ||||||
* / / | Fixed parser treating compound assignment the same as assignment which gave ↵ | unknown | 2019-06-26 | 1 | -2/+1 | |
|/ / | | | | | | | wrong argument usage count | |||||
* | | Merge pull request #29306 from qarmin/small_code_fixes | Rémi Verschelde | 2019-06-12 | 1 | -1/+0 | |
|\ \ | | | | | | | Small fixes to unrechable code, possibly overflows, using NULL pointers | |||||
| * | | Small fixes to unrechable code, possibly overflows, using NULL pointers | qarmin | 2019-06-03 | 1 | -1/+0 | |
| | | | ||||||
* | | | Merge pull request #29678 from akien-mga/err-macros-semicolon | Rémi Verschelde | 2019-06-11 | 1 | -1/+1 | |
|\ \ \ | | | | | | | | | Fix error macro calls not ending with semicolon | |||||
| * | | | Fix error macro calls not ending with semicolon | Rémi Verschelde | 2019-06-11 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's not necessary, but the vast majority of calls of error macros do have an ending semicolon, so it's best to be consistent. Most WARN_DEPRECATED calls did *not* have a semicolon, but there's no reason for them to be treated differently. | |||||
* | | | | Added export hint to 2D and 3D physics and render layers. | MrCdK | 2019-06-07 | 1 | -0/+44 | |
|/ / / | ||||||
* | | | Merge pull request #29433 from bojidar-bg/29406-dictionary-constant | Rémi Verschelde | 2019-06-03 | 1 | -0/+4 | |
|\ \ \ | | | | | | | | | Properly catch certain errors in Dictionary and Array declarations | |||||
| * | | | Properly catch certain errors in Dictionary declarations | Bojidar Marinov | 2019-06-03 | 1 | -0/+4 | |
| |/ / | | | | | | | | | | Fixes #29406 | |||||
* | | | Merge pull request #28099 from lupoDharkael/fix-completion | Rémi Verschelde | 2019-06-03 | 1 | -1/+1 | |
|\ \ \ | |/ / |/| | | Fix code completion not working with class_name | |||||
| * | | Don't allow class_name in built-in scripts | lupoDharkael | 2019-04-17 | 1 | -1/+1 | |
| | | | ||||||
* | | | Merge pull request #27987 from lupoDharkael/classname-duplicate | Rémi Verschelde | 2019-05-28 | 1 | -1/+7 | |
|\ \ \ | | | | | | | | | Prevent using multiple class_name in a class | |||||
| * | | | Prevent using multiple class_name in a class | lupoDharkael | 2019-04-16 | 1 | -1/+7 | |
| | | | | ||||||
* | | | | Fix parse error when extends from subclass of named classes | Geequlim | 2019-05-24 | 1 | -3/+4 | |
| | | | | ||||||
* | | | | Fix typos with codespell | Rémi Verschelde | 2019-05-19 | 1 | -5/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using codespell 1.15.0. Method: ``` $ cat > ../godot-word-whitelist.txt << EOF ang curvelinear doubleclick leapyear lod merchantibility nd numer ois ony que seeked synching te uint unselect webp EOF $ codespell -w -q 3 -I ../godot-word-whitelist.txt --skip="./thirdparty,*.po" $ git diff // undo unwanted changes ``` | |||||
* | | | | Fix typed arrays and dictionaries getting their values shared | Bojidar Marinov | 2019-05-02 | 1 | -29/+16 | |
| | | | | ||||||
* | | | | Merge pull request #27676 from qarmin/small_fixes_2 | Rémi Verschelde | 2019-05-01 | 1 | -9/+4 | |
|\ \ \ \ | |_|/ / |/| | | | Small fixes to static analyzer bugs | |||||
| * | | | Small fixes to static analyzer bugs | qarmin | 2019-04-04 | 1 | -9/+4 | |
| | | | | ||||||
* | | | | GDScript: add variable shadowing warning | lupoDharkael | 2019-04-15 | 1 | -0/+16 | |
| |/ / |/| | | ||||||
* | | | Merge pull request #27863 from bojidar-bg/27460-constant-class-clash | Rémi Verschelde | 2019-04-12 | 1 | -0/+33 | |
|\ \ \ | | | | | | | | | Check subclasses too when checking for name clashes | |||||
| * | | | Check for subclasses when checking for name clashes | Bojidar Marinov | 2019-04-10 | 1 | -0/+33 | |
| | | | | | | | | | | | | | | | | Fixes #27460 | |||||
* | | | | GDScript: Don't allow built-in scripts to use class_name | George Marques | 2019-04-10 | 1 | -0/+4 | |
|/ / / | ||||||
* | | | Style: Apply new changes from clang-format 8.0 | Rémi Verschelde | 2019-04-09 | 1 | -5/+12 | |
| | | | | | | | | | | | | | | | | | | It seems to stay compatible with formatting done by clang-format 6.0 and 7.0, so contributors can keep using those versions for now (they will not undo those changes). | |||||
* | | | Fix -Wimplicit-fallthrough warnings from GCC 8 | Rémi Verschelde | 2019-04-05 | 1 | -6/+12 | |
|/ / | | | | | | | | | | | | | | | | | | | | | | | Adds `FALLTHROUGH` macro to specify when a fallthrough is intentional. Can be replaced by `[[fallthrough]]` if/when we switch to C++17. The warning is now enabled by default for GCC on `extra` warnings level (part of GCC's `-Wextra`). It's not enabled in Clang's `-Wextra` yet, but we could enable it manually once we switch to C++11. There's no equivalent feature in MSVC for now. Fixes #26135. | |||||
* | | Merge pull request #26097 from ↵ | Rémi Verschelde | 2019-04-01 | 1 | -2/+4 | |
|\ \ | | | | | | | | | | | | | danielspaniol/25955-wrong-unreachable-warning-after-returning-from-matchs-wildcard-pattern Fix: Wrong unreachable warning after returning from match's wildcard pattern #25955 | |||||
| * | | Require `return` in all match branches | Daniel Spaniol | 2019-02-20 | 1 | -2/+4 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before the parser only checked if the catch-all branch has a return in order to determine if the entire match block has a return. This code block was assumed to always return. match value: "test": print("test") _: return Now as soon as one of the branches has no return, the entire match block is marked to not have a return. | |||||
* | | | Fix duplicated lines in GDScript bytecode | Bojidar Marinov | 2019-03-14 | 1 | -1/+3 | |
| | | | | | | | | | | | | Fixes #26789 | |||||
* | | | Fix enums coming from other classes without preload | Bojidar Marinov | 2019-03-05 | 1 | -0/+12 | |
| | | | | | | | | | | | | Fix #19704, fix #26001 |