X Tutup
Skip to content

GDScript: Remove some unnecessary booleans#98061

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
dalexeev:gds-remove-unnecessary-bools
Oct 2, 2025
Merged

GDScript: Remove some unnecessary booleans#98061
Repiteo merged 1 commit intogodotengine:masterfrom
dalexeev:gds-remove-unnecessary-bools

Conversation

@dalexeev
Copy link
Member

  1. Remove GDScriptDataType::has_type and replace it with kind = VARIANT. This is cleaner because it allows us to avoid inconsistent states.
  2. Remove unused GDScriptInstance::base_ref_counted field. If we need this again for some reason, there are the owner and script fields.

@dalexeev dalexeev added this to the 4.4 milestone Oct 10, 2024
@dalexeev dalexeev requested a review from a team as a code owner October 10, 2024 15:16
@dalexeev dalexeev marked this pull request as draft February 16, 2025 19:51
@dalexeev dalexeev modified the milestones: 4.4, 4.5 Feb 16, 2025
@dalexeev dalexeev modified the milestones: 4.5, 4.6 Aug 19, 2025
@dalexeev dalexeev force-pushed the gds-remove-unnecessary-bools branch from 1611669 to 72b9f99 Compare October 1, 2025 14:04
@dalexeev dalexeev marked this pull request as ready for review October 1, 2025 14:30
@HolonProduction
Copy link
Member

It might make sense to include the additional parameter removal from #110692

Co-authored-by: HolonProduction <holonproduction@gmail.com>
@dalexeev dalexeev force-pushed the gds-remove-unnecessary-bools branch from 72b9f99 to 5f0ecf4 Compare October 2, 2025 10:39
@dalexeev
Copy link
Member Author

dalexeev commented Oct 2, 2025

@HolonProduction Done. I've included your changes and listed you as a co-author.

Copy link
Member

@HolonProduction HolonProduction left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (Can I approve, if I'm listed as coauthor?)

@Repiteo Repiteo merged commit 9281613 into godotengine:master Oct 2, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Oct 2, 2025

Thanks!

@dalexeev dalexeev deleted the gds-remove-unnecessary-bools branch October 2, 2025 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

X Tutup