-
-
Notifications
You must be signed in to change notification settings - Fork 21k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
52499 preload parsing error when newline encountered #52520
Closed
kdiduk
wants to merge
4,883
commits into
godotengine:master
from
kdiduk:52499-preload-parsing-error-when-newline-encountered
Closed
52499 preload parsing error when newline encountered #52520
kdiduk
wants to merge
4,883
commits into
godotengine:master
from
kdiduk:52499-preload-parsing-error-when-newline-encountered
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Portal autolinking was previously agnostic to room priorities, which meant that portals would link to the first room they found (often outside rooms). This PR fixes this by making the autolinking priority aware, and will preferentially link to internal rooms.
The checking for link room IDs was checking for less than size(), but was not correctly checking for -1, and therefore reading outside the array range. This PR fixes this.
Add support for Sprite3D and animated Sprite3D.
…dary_pvs Portals - Fix secondary PVS bug
…internal Portals - fix autolinking to internal rooms
…rash Portals - fix crash when logging link room names
Portals - add support for Sprite3D
[3.x] Fix various i18n failures
Fixed a bug in the complex PVS generation which was causing recursive loop. Move some of the settings out of RoomManager into Project Settings. Allow PVS generation method to be selected from Project Settings, and control PVS logging.
…ings Portals - fix PVS generation and move settings
…_mode_3.x [3.x] Fix a default shader specular render mode to `SCHLICK_GGX`
Used to specify the expected "CN" in the server certificate if different from the server address.
[Net] Add "dtls_hostname" property to ENet.
Color artifacts could be visible when using negative lights with the Filmic and ACES tonemapping operators, as these did not clamp negative colors.
When synchronizing KinematicBody motion with moving the platform using direct body state, only the linear velocity was taken into account. This change exposes velocity at local point in direct body state and uses it in move_and_slide to get the proper velocity that includes rotations.
…orm-rotation-3.x [3.x] Fix applied rotation from moving platforms in move_and_slide
Same thing that was already done in 2D, applies moving platform motion by using a call to move_and_collide that excludes the platform itself, instead of making it part of the body motion. Helps with handling walls and slopes correctly when the character walks on the moving platform. Also made some minor adjustments to the 2D version and documentation. Co-authored-by: fabriceci <fabricecipolla@gmail.com>
…late Allow unclamped colors in Sprite3D
…sion [3.x] Add horizon specular occlusion
…ive-colors-3.x Clamp negative colors regardless of the tonemapper to avoid artifacts
…ve-sharpening-3.x Add support for contrast-adaptive sharpening in 3D (GLES3 only)
Added one more warning to the hideable warnings. These seem to be benign warnings and are hidden during use in rooms and portals. When used from other areas, only one warning is displayed per run, instead of for every occurrence.
I'm sorry for this 😞 I made this PR by mistake (getting used to GitHub interface). This PR is correct: #52521 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR fixes the issue #52499. The fix is very simple and safe - we just skip newline tokens if we encounter them after the '(' and before the ')'.
This fix is intended only for 3.4 version. The version 4.0 doesn't have this problem since all the parsing code is rewritten.