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

Some more small adjustments for empty strings #21474

Merged
merged 4 commits into from
Feb 13, 2024

Conversation

rettinghaus
Copy link
Contributor

No description provided.

@Jojo-Schmitz
Copy link
Contributor

Jojo-Schmitz commented Feb 12, 2024

See #21445 (comment) ff.

@@ -2014,12 +2014,12 @@ void ExportMusicXml::barlineRight(const Measure* const m, const track_idx_t stra
// note: use barlinetype as found in multi measure rest for last measure of replaced sequence
BarLineType bst = m == mmRLst ? mmR1->endBarLineType() : m->endBarLineType();
const bool visible = m->endBarLineVisible();
String color = u"";
String color = String();
Copy link
Contributor

Choose a reason for hiding this comment

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

better just String color;

@igorkorsukov igorkorsukov merged commit e044aaf into musescore:master Feb 13, 2024
11 checks passed
@rettinghaus rettinghaus deleted the xml/string-check branch February 13, 2024 08:16
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Feb 13, 2024
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Feb 13, 2024
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Feb 13, 2024
Backport of musescore#21474 (and some more)

Co-authored-by: Klaus Rettinghaus <klaus.rettinghaus@gmail.com>
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Apr 9, 2024
Backport of musescore#21474 (and some more)

Co-authored-by: Klaus Rettinghaus <klaus.rettinghaus@gmail.com>
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.

None yet

3 participants