Skip to content

Conversation

@add-uos
Copy link
Contributor

@add-uos add-uos commented Dec 26, 2025

Replace runtime map population with static const translation table using QT_TR_NOOP for better performance and maintainability in CPU device manager

log: optimize CPU number translation handling
bug: https://pms.uniontech.com/bug-view-344133.html https://pms.uniontech.com/bug-view-334225.html

Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Sorry @add-uos, your pull request is larger than the review limit of 150000 diff characters

https://pms.uniontech.com/bug-view-346127.htmle functionality
Add support for enabling/disabling USB keyboard devices through udev rules.
The implementation includes:

- Backend service method to create/remove udev rules for keyboard control
- Device management logic to handle keyboard enable/disable operations
- DBus interface for communication between frontend and backend
- VID/PID validation and sanitization for USB device identification
- Authorization checks to ensure privileged operations are protected

log: add keyboard enable/disable functionality
bug: https://pms.uniontech.com/bug-view-346127.html
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: add-uos, lzwind

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@add-uos
Copy link
Contributor Author

add-uos commented Dec 30, 2025

/forcemerge

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Dec 30, 2025

This pr force merged! (status: unstable)

@deepin-bot deepin-bot bot merged commit 3d0e674 into linuxdeepin:master Dec 30, 2025
16 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants