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

[NUI] Support RemoveIdle API for NUIApplication + Use unified idler callback #6523

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

hinohie
Copy link
Contributor

@hinohie hinohie commented Dec 18, 2024

Let we keep Idler callback list as membery of internal Application class, and allow to remove them during idler callback execute.

Changed Behavior

  • We don't garentee that Delegate function emit time matched with AddIdle function call.
  • We don't allow to add same Delegate function multiple times. Until now, Delegate emitted as the number of AddIdle function call.

@TizenAPI-Bot
Copy link
Collaborator

Build Error:

src/Tizen.NUI/src/internal/Application/Application.cs(398,25): error CS0149: Method name expected [src/Tizen.NUI/Tizen.NUI.csproj]

@hinohie hinohie force-pushed the remove_idle_support branch from d266bf2 to c870f32 Compare December 18, 2024 05:28
@TizenAPI-Bot
Copy link
Collaborator

Internal API Changed

Added: 1, Removed: 0, Changed: 0

Added

+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIApplication::RemoveIdle(System.Delegate)

@TizenAPI-Bot
Copy link
Collaborator

Internal API Changed

Added: 1, Removed: 0, Changed: 0

Added

+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIApplication::RemoveIdle(System.Delegate)

@hinohie hinohie force-pushed the remove_idle_support branch from f8a67a4 to 4d9c231 Compare December 18, 2024 05:54
@TizenAPI-Bot
Copy link
Collaborator

Internal API Changed

Added: 1, Removed: 0, Changed: 0

Added

+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIApplication::RemoveIdle(System.Delegate)

…allback

Let we keep Idler callback list as membery of internal Application class,
and allow to remove them during idler callback execute.

Signed-off-by: Eunki, Hong <[email protected]>
@hinohie hinohie force-pushed the remove_idle_support branch from 4d9c231 to 129f533 Compare December 18, 2024 06:00
@TizenAPI-Bot
Copy link
Collaborator

Internal API Changed

Added: 1, Removed: 0, Changed: 0

Added

+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Void Tizen.NUI.NUIApplication::RemoveIdle(System.Delegate)

Copy link
Contributor

@dongsug-song dongsug-song left a comment

Choose a reason for hiding this comment

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

approved

Copy link
Contributor

@jaehyun0cho jaehyun0cho left a comment

Choose a reason for hiding this comment

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

LGTM

@hinohie hinohie merged commit 27baefc into Samsung:DevelNUI Dec 19, 2024
3 checks passed
@hinohie hinohie deleted the remove_idle_support branch December 19, 2024 01:40
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