Skip to content

Skip video partition on redump-style images#46

Closed
18woldemar wants to merge 7 commits into
XboxDev:masterfrom
18woldemar:master
Closed

Skip video partition on redump-style images#46
18woldemar wants to merge 7 commits into
XboxDev:masterfrom
18woldemar:master

Conversation

@18woldemar

Copy link
Copy Markdown

No description provided.

@InternalLoss

Copy link
Copy Markdown

I can't comment on the code style itself but I can confirm this works flawlessly on macOS (Thank you so much!)

@GXTX

GXTX commented Feb 6, 2021

Copy link
Copy Markdown
Contributor

Split this into 2 PRs please. One for the offset change and the other for adding Android support (although I don't know if we should bother with that).

Comment thread extract-xiso.c
* OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
* ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*\

@GXTX GXTX Feb 6, 2021

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Leave this as is, we have a open PR to address this with #41

Comment thread extract-xiso.c


#define exiso_version "2.7.1 (01.11.14)"
#define exiso_version "2.7.2 (02.04.21)"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Don't change this.

@dracc

dracc commented Feb 6, 2021

Copy link
Copy Markdown
Contributor

Changes look good, but I'd prefer to see them split into separate pull requests as they seem to be quite different in nature.

The splits I'm thinking would be reasonable;
Removing changelog (I'm personally not a fan and believe they've been left there for a reason)
Adding Android support
Adding redump style support

@LukeUsher

Copy link
Copy Markdown
Contributor

Closing this as it is (for the most part) taken care of by #47 and #41

@LukeUsher LukeUsher closed this Jun 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants