MRI: Stub out Graphics.play_movie #138

Merged
mook merged 1 commits from graphics-playmovie-stub into master 2015-10-30 15:20:47 +00:00
mook commented 2015-10-29 05:17:04 +00:00 (Migrated from github.com)

Some people are using it to detect the RGSS version. (Line 52 in this script.)

Obviously this doesn't actually play any movies, but it's better than crashing :)

Let me know if I'm making too many PRs or whatever :)

Some people are using it to detect the RGSS version. ([Line 52 in this script](http://a1tktk.web.fc2.com/vxa_scripts/a1_common_script.txt).) Obviously this doesn't actually play any movies, but it's better than crashing :) Let me know if I'm making too many PRs or whatever :)
Ancurio commented 2015-10-29 15:55:15 +00:00 (Migrated from github.com)

Let me know if I'm making too many PRs or whatever :)

What? No way, contributions are always very welcome! Although the codebase is badly documented so it might unfortunately deter some :)

Also, having separate PRs for fixes towards unrelated parts of the code is definitely the right way to do it.

> Let me know if I'm making too many PRs or whatever :) What? No way, contributions are always very welcome! Although the codebase is badly documented so it might unfortunately deter some :) Also, having separate PRs for fixes towards unrelated parts of the code is definitely the right way to do it.
mook commented 2015-10-30 03:50:38 +00:00 (Migrated from github.com)

Stubbed out in graphics.cpp instead, as requested in 8bd1d9b49d. (Sorry, force push erased it from the PR.) Also, fixed a problem where I was accidentally exposing it in RGSS2 (which sort of defeats the purpose of the script this was trying to fix). Thanks for the feedback!

Stubbed out in `graphics.cpp` instead, as requested in https://github.com/mook/mkxp/commit/8bd1d9b49d1fcad5a5b0beb752535836f80fbf95. (Sorry, force push erased it from the PR.) Also, fixed a problem where I was accidentally exposing it in RGSS2 (which sort of defeats the purpose of the script this was trying to fix). Thanks for the feedback!
Ancurio commented 2015-10-30 09:36:21 +00:00 (Migrated from github.com)

Ah, right, the force push causes the comments to become disattached from this PR. But no worries, force-pushing is a lot better than what I think github wants us to do (endlessly append "fix" commits onto PRs), because the final commit history stays clean.

Other than the minor tab nitpick, it looks great!

Ah, right, the force push causes the comments to become disattached from this PR. But no worries, force-pushing is a lot better than what I think github wants us to do (endlessly append "fix" commits onto PRs), because the final commit history stays clean. Other than the minor tab nitpick, it looks great!
mook commented 2015-10-30 15:03:24 +00:00 (Migrated from github.com)

Tab thing fixed. Commented because IIRC github doesn't notify on forced push to a PR.

Tab thing fixed. Commented because IIRC github doesn't notify on forced push to a PR.
Ancurio commented 2015-10-30 15:21:03 +00:00 (Migrated from github.com)

Thanks for your work!

Thanks for your work!
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: MapleShrine/mkxp#138
No description provided.