-
Notifications
You must be signed in to change notification settings - Fork 35
Add DWI Saving Support + Fix DWI Loading #278
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
Closed
SilentMystification
wants to merge
8
commits into
uvcat7:release
from
SilentMystification:feature/add-dwi-saving-support
Closed
Changes from all commits
Commits
Show all changes
8 commits
Select commit
Hold shift + click to select a range
52c469f
Initial commit
SilentMystification d2662e4
Fix <>
SilentMystification 7a133f5
Update SaveDwi.cpp
SilentMystification 778f2bf
Remove example files
SilentMystification 5718bc3
Whitespace update
SilentMystification d75f76b
Fix DWI loading bug where 64th notes were not being read correctly
SilentMystification 2b52feb
improve quantization by making use of all bracket types
SilentMystification 903f1b4
minor cleanup
SilentMystification File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -199,12 +199,12 @@ static bool ParseNotes(Simfile* sim, char* p, int numPads, int numCols, const ch | |
| case '\r': ++n; break; | ||
| case '(': quantization = 12; ++n; break; | ||
| case '[': quantization = 8; ++n; break; | ||
| case '{': quantization = 4; ++n; break; | ||
| case '{': quantization = 3; ++n; break; | ||
| case '`': quantization = 1; ++n; break; | ||
| case ')': | ||
| case ']': | ||
| case '}': | ||
| case '\'': quantization = 32; ++n; break; | ||
| case '\'': quantization = 24; ++n; break; | ||
| default: | ||
| n = ReadNoteRow(n, chart->notes, row, map, holds, quantization); | ||
|
Owner
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There's actually another bug here, though it would only have subtle effects when editing and copy/pasting: ReadNoteRow should be called with 192/quantization instead of quantization. |
||
| row += quantization; | ||
|
|
@@ -349,6 +349,16 @@ static void ParseTag(Simfile* sim, String tag, char* val) | |
| { | ||
| ParseDisplayBpm(sim->tempo, val); | ||
| } | ||
| else if(tag == "SAMPLESTART") | ||
| { | ||
| double v; | ||
| if(ParseVal(val, v)) sim->previewStart = v; | ||
| } | ||
| else if(tag == "SAMPLELENGTH") | ||
| { | ||
| double v; | ||
| if(ParseVal(val, v)) sim->previewLength = v; | ||
| } | ||
| } | ||
|
|
||
| // =================================================================================== | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe this makes sense as a default shortcut as many other programs use this exact combination to initiate the Save As flow.