Skip to content

[code suggestion] GetConfigured*EnabledValue()GetConfiguredEnabledValue(variable) #29360

@Jay-o-Way

Description

Description of the new feature / enhancement

Idea came to me via pr #29325 at https://github.com/microsoft/PowerToys/pull/29325/files#r1367455714

Here's a list of things like GetConfiguredAlwaysOnTopEnabledValue() for every module. This looks very redundant and uneasy to modify when a module needs to be added (or removed). I was thinking a function/method like GetConfiguredEnabledValue("AlwaysOnTop") would be better, because then we can use (an array with) module names to loop through. This will allow to simplify or "compress" lots of code.

At the very least it will also be more readable :)

Scenario when this would be used?

Smooth code?

Supporting information

No response

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Area-BuildIssues pertaining to the build system, CI, infrastructure, metaIdea-EnhancementNew feature or request on an existing product

    Projects

    • Status

      Waiting for input ⏳

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions