| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
| |
This partially addresses
https://github.com/godotengine/godot-proposals/issues/1014.
|
|
|
|
|
|
|
|
|
|
|
| |
ToolButton has no redeeming differences with Button;
it's just a Button with the Flat property enabled by default.
Removing it avoids some confusion when creating GUIs.
Existing ToolButtons will be converted to Buttons, but the Flat
property won't be enabled automatically.
This closes https://github.com/godotengine/godot-proposals/issues/1081.
|
|
|
|
|
| |
Using clang-tidy's `readability-braces-around-statements`.
https://clang.llvm.org/extra/clang-tidy/checks/readability-braces-around-statements.html
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I couldn't find a tool that enforces it, so I went the manual route:
```
find -name "thirdparty" -prune \
-o -name "*.cpp" -o -name "*.h" -o -name "*.m" -o -name "*.mm" \
-o -name "*.glsl" > files
perl -0777 -pi -e 's/\n}\n([^#])/\n}\n\n\1/g' $(cat files)
misc/scripts/fix_style.sh -c
```
This adds a newline after all `}` on the first column, unless they
are followed by `#` (typically `#endif`). This leads to having lots
of places with two lines between function/class definitions, but
clang-format then fixes it as we enforce max one line of separation.
This doesn't fix potential occurrences of function definitions which
are indented (e.g. for a helper class defined in a .cpp), but it's
better than nothing. Also can't be made to run easily on CI/hooks so
we'll have to be careful with new code.
Part of #33027.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Which means that reduz' beloved style which we all became used to
will now be changed automatically to remove the first empty line.
This makes us lean closer to 1TBS (the one true brace style) instead
of hybridating it with some Allman-inspired spacing.
There's still the case of braces around single-statement blocks that
needs to be addressed (but clang-format can't help with that, but
clang-tidy may if we agree about it).
Part of #33027.
|
| |
|
|\
| |
| | |
Use a different icon for the debugger tab with both warnings and errors
|
| |
| |
| |
| |
| | |
This makes it possible to see if both errors and warnings were
pushed without having to open the tab.
|
|/
|
|
|
|
|
| |
Part of #33027, also discussed in #29848.
Enforcing the use of brackets even on single line statements would be
preferred, but `clang-format` doesn't have this functionality yet.
|
| |
|
| |
|
| |
|
|
|
|
| |
Rename editor plugins to match the new node names.
|
|
|
|
| |
Fixes #30736.
|
| |
|
| |
|
| |
|
|
|
|
| |
Fix: #37121
|
|
|
|
|
|
|
|
|
|
| |
For us, it practically only changes the fact that `A<A<int>>` is now
used instead of the C++03 compatible `A<A<int> >`.
Note: clang-format 10+ changed the `Standard` arguments to fully
specified `c++11`, `c++14`, etc. versions, but we can't use `c++17`
now if we want to preserve compatibility with clang-format 8 and 9.
`Cpp11` is still supported as deprecated alias for `Latest`.
|
|
|
| |
Found via `codespell`
|
|
|
|
|
| |
Shared RemoteDebuggerPeer code between client and server.
Move editor profilers inside editor/debugger folder.
|
| |
|
|
|
|
|
|
| |
EditorDebuggerInspector is in tree, so it gets automatically deleted,
when clearing errors the debugger should not fake a process
notification.
|
|
|
|
|
|
|
|
|
| |
Remove now unnecessary bindings of signal callbacks in the public API.
There might be some false positives that need rebinding if they were
meant to be public.
No regular expressions were harmed in the making of this commit.
(Nah, just kidding.)
|
|
|
| |
The original change was in #36340
|
| |
|
|
|