Skip to content
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

Remove methods in header removed in c7eb814 #74898

Merged
merged 1 commit into from
Mar 16, 2023

Conversation

Spartan322
Copy link
Contributor

@Spartan322 Spartan322 commented Mar 14, 2023

See #67888
c7eb814 missed these methods in the RichTextLabel's header
Remove set_override_selected_font_color from rich_text_label.h
Remove is_override_selected_font_color from rich_text_label.h
c7eb814 missed these methods in the TextEdit's header
Remove set_override_selected_font_color from text_edit.h
Remove is_override_selected_font_color from text_edit.h

As it stands, without a source file, all these methods can accomplish in this state is erroring in linkage if used anyway. Noticed this on one of my PRs and was curious why I wasn't getting an intellisense error but got a linkage error, this prevents that strange case.

@Spartan322 Spartan322 requested a review from a team as a code owner March 14, 2023 08:03
@Chaosus Chaosus added this to the 4.1 milestone Mar 14, 2023
@clayjohn clayjohn requested review from KoBeWi and bruvzg March 15, 2023 23:45
@KoBeWi
Copy link
Member

KoBeWi commented Mar 15, 2023

text_edit.h also has a leftover.

c7eb814 missed these methods in the RichTextLabel's header
Remove set_override_selected_font_color from rich_text_label.h
Remove is_override_selected_font_color from rich_text_label.h
c7eb814 missed these methods in the TextEdit's header
Remove set_override_selected_font_color from text_edit.h
Remove is_override_selected_font_color from text_edit.h
@Spartan322
Copy link
Contributor Author

text_edit.h has been added.

@YuriSizov YuriSizov merged commit a1c2f03 into godotengine:master Mar 16, 2023
@YuriSizov
Copy link
Contributor

Thanks!

@Spartan322 Spartan322 deleted the patch-1 branch March 16, 2023 12:49
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.

4 participants