Skip to content
This repository has been archived by the owner on Apr 15, 2023. It is now read-only.

TVHeadend compile fixes for Windows #414

Closed
wants to merge 2 commits into from
Closed

Conversation

margro
Copy link

@margro margro commented Feb 7, 2012

Hi Lars,

A few compile fixes for the tvheadend addon under Windows (VS2010).
Please check if you agree with 790f448.

Grt,
Marcel

Solves the "error C2632: 'int' followed by 'int' is illegal" error due to a double typedef of ssize_T
@margro
Copy link
Author

margro commented Feb 7, 2012

Hmmz, VS2010 has its own stdint.h file which is different than the one in lib/platform/windows/stdint.h. Thhe latter might be unneeded

@opdenkamp
Copy link
Owner

hmm odd, both are building fine here on windows. that stdint.h is needed on vs2008. xbmc doesn't target that anymore, but I'd written lib/platform for libCEC, which does use vs2008 to build the .net wrapper.

I could just remove lib/platform/windows/stdint.h from xbmc instead too I suppose

@margro
Copy link
Author

margro commented Feb 8, 2012

Ok. Strange. I will do a fresh clone of your master branch and try it again. I got the compile errors after merging your master into mine.

Removing the stdint.h header only for xbmc is ok.

@margro
Copy link
Author

margro commented Feb 8, 2012

The result on my machine of a debug build of a clean clone from opdenkamp/master 0c5dd3d:

1>------ Build started: Project: pvrclient_tvheadend, Configuration: Debug Win32 ------
1>  client.cpp
1>e:\src\github\opdenkamp\xbmc\lib\libhts\win32\include\stdint.h(59): error C2371: 'int8_t' : redefinition; different basic types
1>          e:\src\github\opdenkamp\xbmc\lib\platform\windows\os-types.h(58) : see declaration of 'int8_t'
1>  HTSPDemux.cpp
1>e:\src\github\opdenkamp\xbmc\lib\ffmpeg\libavutil\common.h(170): warning C4244: 'return' : conversion from 'int64_t' to 'int32_t', possible loss of data
1>e:\src\github\opdenkamp\xbmc\lib\platform\windows\os-types.h(58): error C2371: 'int8_t' : redefinition; different basic types
1>          e:\src\github\opdenkamp\xbmc\lib\libhts\win32\include\stdint.h(59) : see declaration of 'int8_t'
1>e:\src\github\opdenkamp\xbmc\lib\platform\windows\os-types.h(71): error C2632: 'int' followed by 'int' is illegal
1>e:\src\github\opdenkamp\xbmc\lib\platform\windows\os-types.h(71): warning C4091: 'typedef ' : ignored on left of '__w64 int' when no variable is declared
1>  HTSPData.cpp
1>e:\src\github\opdenkamp\xbmc\lib\libhts\win32\include\stdint.h(59): error C2371: 'int8_t' : redefinition; different basic types
1>          e:\src\github\opdenkamp\xbmc\lib\platform\windows\os-types.h(58) : see declaration of 'int8_t'
1>  HTSPConnection.cpp
1>e:\src\github\opdenkamp\xbmc\lib\platform\windows\os-types.h(58): error C2371: 'int8_t' : redefinition; different basic types
1>          e:\src\github\opdenkamp\xbmc\lib\libhts\win32\include\stdint.h(59) : see declaration of 'int8_t'
1>  Generating Code...
========== Build: 0 succeeded, 1 failed, 43 up-to-date, 0 skipped ==========

@opdenkamp
Copy link
Owner

right, a clean build does the same here now.
I've fixed it in libcec by including stdint.h. I'll sync it again and pick 9290cd2

@opdenkamp
Copy link
Owner

fixed

@opdenkamp opdenkamp closed this Feb 9, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants