Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TechDebt] Ret. Move the code for modules into separate files #27523

Open
Eism opened this issue Apr 3, 2025 · 0 comments
Open

[TechDebt] Ret. Move the code for modules into separate files #27523

Eism opened this issue Apr 3, 2025 · 0 comments
Assignees

Comments

@Eism
Copy link
Contributor

Eism commented Apr 3, 2025

Problem:
The Ret class, which is located in the framework, knows about modules that are not located in the framework.

enum class Code {
        Undefined       = -1,
        Ok              = 0,
        UnknownError    = 1,
...
        ProjectFirst  = 3000,
        ProjectLast   = 3999,
...
    };

Solution:
create two files:

  • for framework. List a range for all modules from the framework. We can leave all the code for the modules from the framework in Ret. It is important to set the Last value, for example, equal to 99,999
  • for application. List a range for all application modules. Start should be static_cast(Code::Last) + 1
@Eism Eism added the tech debt label Apr 3, 2025
@Eism Eism self-assigned this Apr 3, 2025
@bkunda bkunda moved this to To do in MuseScore Studio 4.6 Apr 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: To do
Development

No branches or pull requests

1 participant