Skip to content

Added get_path method to Run so the path can be actually accessed.#312

Merged
CryZe merged 2 commits intoLiveSplit:masterfrom
Ynscription:accessPathFromRun
Mar 12, 2020
Merged

Added get_path method to Run so the path can be actually accessed.#312
CryZe merged 2 commits intoLiveSplit:masterfrom
Ynscription:accessPathFromRun

Conversation

@Ynscription
Copy link
Copy Markdown
Contributor

Aded a get_path method to Run so the path can be accessed for saving, as I asume it was intended to be.

The Run struct contains an Option to store the path to the save file it was loaded from.
This comes in really handy when saving, as the path can be simply read from the path.
However, the path attribute of Run was private and inaccessible in any way, making it unusable.

Now the path should be easily accessible.

Copy link
Copy Markdown
Collaborator

@CryZe CryZe left a comment

Choose a reason for hiding this comment

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

Also there's a formatting mismatch. Probably a tab or some weird whitespace.

@CryZe CryZe added code quality Affects the quality of the code. enhancement An improvement for livesplit-core. labels Mar 12, 2020
@CryZe CryZe merged commit 1027a92 into LiveSplit:master Mar 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code quality Affects the quality of the code. enhancement An improvement for livesplit-core.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants