RDKEMW-12937: Test PR to validate workflow#10
Conversation
| #include <mutex> | ||
| #include "tracing/Logging.h" | ||
|
|
||
|
|
There was a problem hiding this comment.
This adds an extra blank line between the include block and the namespace, creating a whitespace-only change and making the diff noisier. Consider keeping a single blank line here for consistency with the surrounding style.
|
All contributors have signed the CLA ✍️ ✅ |
| #include <fstream> | ||
| #include "tracing/Logging.h" | ||
|
|
||
|
|
There was a problem hiding this comment.
This adds an extra blank line between the includes and the namespace declaration, creating a whitespace-only change and slightly reducing consistency with other files in this module (which typically have a single blank line here). Consider removing the extra empty line to keep diffs clean and formatting consistent.
| * limitations under the License. | ||
| */ | ||
|
|
||
|
|
There was a problem hiding this comment.
There are now two consecutive blank lines between the license header and the first include. This appears inconsistent with nearby files (e.g., plugin/UserSettings.cpp keeps a single blank line before includes) and adds unnecessary whitespace—please drop the extra empty line.
| #include <fstream> | ||
| #include "tracing/Logging.h" | ||
|
|
||
|
|
There was a problem hiding this comment.
There are now two consecutive blank lines between the include block and the start of the namespace. To keep formatting consistent and reduce diff noise, remove the extra empty line.
No description provided.