Skip to content
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

Fix for iOS-64 compilation issue #55

Merged

Conversation

anna-khakimova
Copy link
Collaborator

/Users/itseez/Projects/VMF/vmf-3.0/vmf/3rdparty/xmp/third-party/zlib/gzlib.c:260:24:
error: implicit declaration of function 'lseek' is invalid in C99

@apavlenko apavlenko changed the title fix MacOS compilation issue Fix for iOS-64 compilation issue Mar 28, 2016
@apavlenko apavlenko added this to the Milestone 2 milestone Mar 28, 2016
@apavlenko apavlenko self-assigned this Mar 28, 2016
@coveralls
Copy link

Coverage Status

Coverage remained the same at 77.74% when pulling f993d59 on anna-plyakina:fix_MACOS_compilation_issue into e724dd4 on apavlenko:vmf-3.0.

@codecov-io
Copy link

Current coverage is 77.74%

Merging #55 into vmf-3.0 will not affect coverage as of 42b56d8

@@            vmf-3.0     #55   diff @@
=======================================
  Files            59      59       
  Stmts          6240    6240       
  Branches          0       0       
  Methods                           
=======================================
  Hit            4851    4851       
  Partial           0       0       
  Missed         1389    1389       

Review entire Coverage Diff as of 42b56d8


Uncovered Suggestions

  1. +0.15% via ...f_MetadataStream.cpp#823...831
  2. +0.13% via ...f_MetadataStream.cpp#785...792
  3. +0.09% via ..._vmf_MetadataSet.cpp#24...28
  4. See 7 more...

Powered by Codecov. Updated on successful CI builds.

@apavlenko
Copy link
Owner

👍

@apavlenko apavlenko merged commit f2e047e into apavlenko:vmf-3.0 Mar 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants