Skip to content
Snippets Groups Projects

Update fopen calls for Windows compatibility

Closed SmilingWolf requested to merge (removed):patch-1 into master

Open files in binary mode, or on Windows fread/fwrite will treat the files as text and mangle the read/written data

Edited by SmilingWolf

Merge request reports

Approved by
Ready to merge by members who can write to the target branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • SmilingWolf added 1 commit

    added 1 commit

    • 0d5697a5 - More fopen(file, "...b") for Windows compatibility

    Compare with previous version

  • SmilingWolf changed title from Update ivf.c for Windows compatibility to Update fopen calls for Windows compatibility

    changed title from Update ivf.c for Windows compatibility to Update fopen calls for Windows compatibility

  • SmilingWolf changed the description

    changed the description

  • Could you please update the patches to have proper authorship (using a real name)?

  • Author Contributor

    That's a bit of a complicated request for me. I've been fairly successfull at keeping the online persona separated from the RL one and I'd like to continue doing so. The nickname and email address associated have been in use for more than 10 years. It's me on BitBucket, GitHub, GitLab, aomedia's googlesource, and a number of online communities (e.g. doom9). If it is a necessity, is there somewhere off the public record where I can sign, similar to what I did with AOM's CLA? Sorry for the trouble

    Edited by SmilingWolf
  • Yes, please mail me.

  • closed

  • Author Contributor

    Nuking this one and going with a new one with the synced master

Please register or sign in to reply
Loading