[NUI] Add diagnostic logs for application initialization delay - #7766
[NUI] Add diagnostic logs for application initialization delay#7766tscholb wants to merge 1 commit into
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds informational logging statements to track the execution progress of the NUICoreBackend.Run() method. The review feedback suggests removing the redundant class and method name prefixes from the log messages, as Tizen.Log.Info automatically captures caller details. Additionally, it is recommended to use Log.Info directly instead of the fully qualified Tizen.Log.Info to maintain consistency with the rest of the file.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| Tizen.Tracer.End(); | ||
|
|
||
| Tizen.Tracer.Begin("[NUI] NUICorebackend Run(): args of main set, window type set"); | ||
| Tizen.Log.Info("NUI", "NUICoreBackend.Run(): begin to set main arguments"); |
There was a problem hiding this comment.
The log message explicitly includes the class and method name (NUICoreBackend.Run(): ). However, Tizen.Log.Info automatically captures and prints the caller's file name, member name, and line number using [CallerFilePath] and [CallerMemberName] attributes. Including this prefix manually is redundant and leads to duplicated information in the log output.\n\nAdditionally, for consistency with the rest of this file, you should use Log.Info directly instead of the fully qualified Tizen.Log.Info.
Log.Info("NUI", "begin to set main arguments");| args[0] = this.GetType().Assembly.FullName.Replace(" ", ""); | ||
| } | ||
|
|
||
| Tizen.Log.Info("NUI", "NUICoreBackend.Run(): main arguments set, begin to create application"); |
There was a problem hiding this comment.
The log message explicitly includes the class and method name (NUICoreBackend.Run(): ). However, Tizen.Log.Info automatically captures and prints the caller's file name, member name, and line number using [CallerFilePath] and [CallerMemberName] attributes. Including this prefix manually is redundant and leads to duplicated information in the log output.\n\nAdditionally, for consistency with the rest of this file, you should use Log.Info directly instead of the fully qualified Tizen.Log.Info.
Log.Info("NUI", "main arguments set, begin to create application");| } | ||
| } | ||
| } | ||
| Tizen.Log.Info("NUI", "NUICoreBackend.Run(): application created"); |
There was a problem hiding this comment.
The log message explicitly includes the class and method name (NUICoreBackend.Run(): ). However, Tizen.Log.Info automatically captures and prints the caller's file name, member name, and line number using [CallerFilePath] and [CallerMemberName] attributes. Including this prefix manually is redundant and leads to duplicated information in the log output.\n\nAdditionally, for consistency with the rest of this file, you should use Log.Info directly instead of the fully qualified Tizen.Log.Info.
Log.Info("NUI", "application created");
|
🤖 [AI Review] Reviewed — no findings. Scope checked:
No 🔴 critical issues, no 🟡 suggestions to flag. Automated review — final merge decision rests with human reviewers. |
2 participants
Description of Change
API Changes