Cleanup HWiNFO Platform

[CasperH2O]

Summary by CodeRabbit

• Refactor
  • Simplified the startup process for the hardware information tool.
  • Improved the watchdog timer behavior for better stability when shared memory is disabled.
  • Streamlined the process initiation sequence for enhanced performance.

[coderabbitai]

Walkthrough

The update to the HandheldCompanion project simplifies the interaction with HWiNFO, a hardware information and diagnostic tool. The changes involve cleaning up the code by removing outdated comments and unused sections related to shared memory features. This indicates a move towards a more streamlined approach in starting the HWiNFO processes and handling shared memory within the application.

Changes

File Path	Change Summary
HandheldCompanion/.../HWiNFO.cs	Simplified Start method, updated Watchdog_Elapsed method for shared memory handling, and removed obsolete shared memory code from StartProcess method.

🐇🌟
Changes in the code, oh so bright,
Clean and swift, like a kite in flight.
No more clutter, no more fuss,
CodeRabbit hopped away, without a cuss. 🚀🌿

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on X ?

---

Tips

Chat with CodeRabbit Bot (@coderabbitai)

• You can reply to a review comment made by CodeRabbit.
• You can tag CodeRabbit on specific lines of code or files in the PR by tagging @coderabbitai in a comment.
• You can tag @coderabbitai in a PR comment and ask one-off questions about the PR and the codebase. Use quoted replies to pass the context for follow-up questions.

CodeRabbit Commands (invoked as PR comments)

• @coderabbitai pause to pause the reviews on a PR.
• @coderabbitai resume to resume the paused reviews.
• @coderabbitai review to trigger a review. This is useful when automatic reviews are disabled for the repository.
• @coderabbitai resolve resolve all the CodeRabbit review comments.
• @coderabbitai help to get help.

Additionally, you can add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Configration File (.coderabbit.yaml)

• You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
• The JSON schema for the configuration file is available here.
• If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/coderabbit-overrides.v2.json

[coderabbitai]

Review Status

Actionable comments generated: 0

Configuration used: CodeRabbit UI

Commits

Files that changed from the base of the PR and between 25cc827 and 9a1d582.

Files selected for processing (1)

• HandheldCompanion/Platforms/HWiNFO.cs (5 hunks)

Additional comments: 5

HandheldCompanion/Platforms/HWiNFO.cs (5)

• 133-140: The cleanup in the Start method appears to have removed some commented-out code without affecting the logic of starting the HWiNFO process. This should improve readability and maintainability.

• 184-192: In the Watchdog_Elapsed method, the catch block has been modified to handle cases where shared memory is disabled. The removal of the commented-out code and the "todo" note seems to streamline the method without changing its behavior.

• 195-201: The logic in the Watchdog_Elapsed method to handle the case where HWiNFO could not be polled remains unchanged. The cleanup has not introduced any new issues here.

• 530-535: The StartProcess method has had some settings properties set, which are unrelated to shared memory. It's important to ensure that these changes are intentional and correctly reflect the desired behavior of the application.

#!/bin/bash
# Verify that the properties set in StartProcess method are intentional and correct.
# Check for any discussions or issues related to these properties.
gh issue list --search "StartProcess and properties set"

• 670-673: The GPUFrequencyChangedHandler delegate has been left untouched, which is expected as the PR focuses on cleaning up shared memory-related code. It's good to see that the cleanup has not inadvertently affected other parts of the code.