summaryrefslogtreecommitdiffstats
path: root/editor/debugger/script_editor_debugger.cpp
Commit message (Collapse)AuthorAgeFilesLines
* Replace flat buttons with flat-styled buttons with a visible pressed stateYuri Sizov2023-09-191-8/+8
|
* Add EditorStringNames singletonkobewi2023-09-031-34/+35
|
* Avoid unnecessary inspector updates when loading or switching scenesYuri Sizov2023-08-121-3/+3
| | | | | | This should result in some noticeable performance improvements, aside from fixing bugs due to conflicts in logic. This also simplifies some related code identified while debugging.
* Merge pull request #79280 from raulsntos/dotnet/better-push-errorRémi Verschelde2023-08-031-3/+9
|\ | | | | | | C#: Improve `GD.PushError` and `GD.PushWarning`
| * C#: Improve `GD.PushError` and `GD.PushWarning`Raul Santos2023-07-111-3/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Use the name, file path and line number of the caller that invokes `GD.PushError` and `GD.PushWarning` instead of the location in the C++ `runtime_interop.cpp` file. - Improvements to getting the C# stack trace. - Use C# type keywords for built-in types in method declarations. - Remove extra space before each parameter in method declarations. - Skip one more frame to avoid `NativeInterop.NativeFuncs`. - Skip methods annotated with the `[StackTraceHidden]` attribute. - Improvements to `ScriptEditorDebugger` when source is in project. - Avoid overriding error metadata when the source is inside the project file. - Use the source function in the title when the source is inside the project file. Users that use these methods would expect the reported location printed by these methods to correspond to a location in their project source files. Specifically, they'd expect to see the file path and line number at which they call these methods, and not the location of the C++ code (which is always the same). Now, these methods are a lot more useful since users can know which line in their source code printed the error/warning.
* | Support threads in the script debuggerJuan Linietsky2023-07-261-69/+138
|/ | | | | | | | | | * This implementation adds threads on the side of the client (script debugger). * Some functions of the debugger are optimized. * The profile is also now thread safe using atomics. * The editor can switch between multiple threads when debugging. This PR adds threaded support for the script language debugger. Every thread has its own thread local data and it will connect to the debugger using multiple thread IDs. This means that, now, the editor can receive multiple threads entering debug mode at the same time.
* Changed error tree initialization to allow reselectionsamar-012023-06-101-0/+1
|
* Rename the Debugger's stack debug section to Stack TraceAaron Franke2023-05-111-1/+1
|
* Improve editor state persistenceHendrik Brucker2023-05-111-0/+8
|
* Improve includes of EditorNode (and everything else)Yuri Sizov2023-04-071-1/+1
| | | | | | Also start organizing editor-specific GUI components into a dedicated folder, `editor/gui`. Also move `editor_file_server` next to the rest of debugger classes.
* Refresh filesystem when saving remote branchkobewi2023-03-241-0/+7
|
* Make sure Script Debugger is updated with the editor themeYuri Sizov2023-03-101-20/+28
|
* Pace debugger draw requestsPedro J. Estébanez2023-02-071-1/+5
|
* Restore missing info in the error tooltipsPaul Joannon2023-01-281-2/+2
|
* Better error display in debugger panelPaul Joannon2023-01-271-7/+31
| | | | | | - Use the right stack frame info as title of the error. - Use the actual C# exception type as error for exceptions raised from C#. - Show the right language instead of always **C++ Error**.
* One Copyright Update to rule them allRémi Verschelde2023-01-051-29/+29
| | | | | | | | | | | | | | | | | | | | As many open source projects have started doing it, we're removing the current year from the copyright notice, so that we don't need to bump it every year. It seems like only the first year of publication is technically relevant for copyright notices, and even that seems to be something that many companies stopped listing altogether (in a version controlled codebase, the commits are a much better source of date of publication than a hardcoded copyright statement). We also now list Godot Engine contributors first as we're collectively the current maintainers of the project, and we clarify that the "exclusive" copyright of the co-founders covers the timespan before opensourcing (their further contributions are included as part of Godot Engine contributors). Also fixed "cf." Frenchism - it's meant as "refer to / see".
* Fix: Profiler and Visual Profiler start/stop state inconsistencystmSi2022-12-231-1/+11
|
* Fix String::word_wrap() for long wordsHaoyu Qiu2022-12-161-1/+10
| | | | | | | | | - Changes `TextServer.string_get_word_breaks()` - Returns pairs of boundary start and end offsets - Accepts `chars_per_line` to return line breaks - Removes `String::word_wrap()` Co-authored-by: bruvzg <7645683+bruvzg@users.noreply.github.com>
* Use forward-declarations in big editor classestrollodel2022-11-291-0/+2
|
* FIx debugger can't inspect sub objects.Zae2022-11-251-0/+2
|
* Merge pull request #68709 from MewPurPur/instance-begone-part3Rémi Verschelde2022-11-161-3/+3
|\ | | | | | | Remove more instances of 'instance' being used as a verb
| * Remove more instances of 'instance' being used as a verbVolTer2022-11-161-3/+3
| |
* | [MP] Move engine and editor profilers to a plugin.Fabio Alessandrelli2022-11-141-23/+1
| | | | | | | | Also refactor the editor plugin out of the ReplicationEditor.
* | [Editor] Better expose EditorDebuggerPlugin.Fabio Alessandrelli2022-11-141-43/+12
|/ | | | | | | | | | | | | | | Now splitted into two classes: - EditorDebuggerPlugin (RefCounted). - EditorDebuggerSession (abstract). This allows the EditorPlugin to be in control of the debugger plugin lifecycle, be notified when sessions are created, and customize each of them independently. We should slowly transition the various profilers and captures in ScriptEditorDebugger to their own plugins, and decouple ScriptEditorDebugger from it's UI part (making it the "real" EditorDebuggerSession potentially dropping the wrappers).
* Unify usage of GLOBAL/EDITOR_GETkobewi2022-10-181-1/+1
|
* Fix MSVC warnings, rename shadowed variables, fix uninitialized values, ↵bruvzg2022-10-071-12/+12
| | | | change warnings=all to use /W4.
* Debugger: Fix fetching source to link C++ error on GitHubRémi Verschelde2022-10-061-1/+15
| | | | Fixes #66974.
* Fix profiler being disabled at wrong timeshalgriffiths2022-09-131-3/+6
| | | | Co-authored by: Will Whitty <tavurth@gmail.com>
* Rename TreeItem's `set_tooltip` to `set_tooltip_text`Micky2022-08-301-2/+2
| | | | | | | | | `set_tooltip` -> `set_tooltip_text` `get_tooltip` -> `get_tooltip_text` For consistency: `get_button_tooltip` -> `get_button_tooltip_text` And the `tooltip` parameter in `add_button` was renamed to `tooltip_text`
* Revert "Remove NOTIFICATION_ENTER_TREE when paired with ↵Rémi Verschelde2022-08-291-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | NOTIFICATION_THEME_CHANGED" This reverts commit 4b817a565cab8af648c88cfc7ab6481e86ee3625. Fixes #64988. Fixes #64997. This caused several regressions (#64988, #64997, https://github.com/godotengine/godot/issues/64997#issuecomment-1229970605) which point at a flaw in the current logic: - `Control::NOTIFICATION_ENTER_TREE` triggers a *deferred* notification with `NOTIFCATION_THEME_CHANGED` as introduced in #62845. - Some classes use their `THEME_CHANGED` to cache theme items in member variables (e.g. `style_normal`, etc.), and use those member variables in `ENTER_TREE`, `READY`, `DRAW`, etc. Since the `THEME_CHANGE` notification is now deferred, they end up accessing invalid state and this can lead to not applying theme properly (e.g. for EditorHelp) or crashing (e.g. for EditorLog or CodeEdit). So we need to go back to the drawing board and see if `THEME_CHANGED` can be called earlier so that the previous logic still works? Or can we refactor all engine code to make sure that: - `ENTER_TREE` and similar do not depend on theme properties cached in member variables. - Or `THEME_CHANGE` does trigger a general UI update to make sure that any bad theme handling in `ENTER_TREE` and co. gets fixed when `THEME_CHANGE` does arrive for the first time. But that means having a temporary invalid (and possibly still crashing) state, and doing some computations twice which might be heavy (e.g. `EditorHelp::_update_doc()`).
* Merge pull request #64885 from Mickeon/rename-tooltip-hintRémi Verschelde2022-08-281-10/+10
|\ | | | | Rename `hint_tooltip` to `tooltip_text` & setter getter
| * Rename `hint_tooltip` to `tooltip_text` & setgetMicky2022-08-271-10/+10
| | | | | | | | | | | | | | | | `hint_tooltip` -> `tooltip_text` `set_tooltip` -> `set_tooltip_text` `_get_tooltip` -> `get_tooltip_text` Updates documentation, too.
* | Remove NOTIFICATION_ENTER_TREE when paired with NOTIFICATION_THEME_CHANGEDAaron Record2022-08-271-2/+2
|/
* Make Debugger/Misc LineEdits uneditablekobewi2022-08-181-0/+3
|
* Extract EditorResourceConversionPlugin into its own source files and clean ↵Yuri Sizov2022-07-311-1/+0
| | | | up editor includes
* Remove Signal connect bindsJuan Linietsky2022-07-291-4/+4
| | | | | Remove the optional argument p_binds from `Object::connect` since it was deprecated by Callable.bind(). Changed all uses of it to Callable.bind()
* Double check the selected tree item on error activated and error selected ↵Hristo Stamenov2022-07-061-0/+9
| | | | callbacks as the value might be nullptr which would cause a crash.
* Fixing Print_rich which only displays correctly in terminalVoylin2022-07-011-0/+3
| | | There was an issue that the type was not passed through correctly. These couple of lines fix this issue and make print_rich work as expected.
* Move duplicate AutoWrap, Overrun and VisibleChar behavior enums to the ↵bruvzg2022-06-161-1/+1
| | | | TextServer.
* Merge pull request #61736 from KoBeWi/final_form_of_the_fixFabio Alessandrelli2022-06-121-0/+2
|\ | | | | Fix Debugger tab not updating when clearing errors
| * Fix Debugger tab not updating when clearing errorskobewi2022-06-061-0/+2
| |
* | i18n: Misc fixes translation stringsRémi Verschelde2022-06-081-1/+1
|/ | | | Adds some translator comments to solve some questions raised on Weblate.
* Use consistent casing in editor filter/search barsFireForge2022-05-281-1/+1
|
* Add the button pressed to some signals in Treetrollodel2022-05-211-4/+12
|
* Replace most uses of Map by HashMapreduz2022-05-161-4/+4
| | | | | | | | | | | | * Map is unnecessary and inefficient in almost every case. * Replaced by the new HashMap. * Renamed Map to RBMap and Set to RBSet for cases that still make sense (order matters) but use is discouraged. There were very few cases where replacing by HashMap was undesired because keeping the key order was intended. I tried to keep those (as RBMap) as much as possible, but might have missed some. Review appreciated!
* Rename profiler "Idle Time" to "Process Time"Hugo Locurcio2022-05-051-4/+4
| | | | | References to "idle time" are progressively being replaced by "process time" throughout the engine to avoid confusion.
* Rename Transform2D "elements" to "columns"Aaron Franke2022-04-291-1/+1
|
* Add an external link editor iconHugo Locurcio2022-04-241-1/+1
| | | | | | | | | This replaces the existing "chainlink" instance icon that was used for external links. That icon is still used for scene instancing. The icon was designed by redlamp. Co-authored-by: Taylor Wright <taylor@redlamp.org>
* Make FileAccess and DirAccess classes reference counted.bruvzg2022-04-111-2/+2
|
* Capitalize names in the debuggerFireForge2022-03-291-2/+3
|