Re: [Mesa-dev] [AppVeyor] mesa master #5788 failed

2017-10-13 Thread Jose Fonseca
Per https://bugs.freedesktop.org/show_bug.cgi?id=103253 it's not just MSVC. So I'd really recommend sticking to one of the standard C types: intptr_t or int64_t. Jose On 13/10/17 10:41, Jose Fonseca wrote: SSIZE_T is included on the headers.  We include it in most places, but not necessary

Re: [Mesa-dev] [AppVeyor] mesa master #5788 failed

2017-10-13 Thread Jose Fonseca
SSIZE_T is included on the headers. We include it in most places, but not necessary all. It brings a lot of baggage, so I don't recommend including it just for a ssize_t defitinion. Instead I'd recommend replacing ssize_t in our code with plain standard intptr_t. If you really rather use

Re: [Mesa-dev] [AppVeyor] mesa master #5788 failed

2017-10-13 Thread Jason Ekstrand
This is aparently because MSVC doesn't have ssize_t. How do you recommend fixing it? I found a stack overflow comment that seems to be related here: https://stackoverflow.com/questions/22265610/why-ssize-t-in-visual-studio-2010-is-defined-as-unsigned The recommendation given in one of the

[Mesa-dev] [AppVeyor] mesa master #5788 failed

2017-10-12 Thread AppVeyor
Build mesa 5788 failed Commit b8ab78d1af by Jason Ekstrand on 10/11/2017 7:13 PM: anv/pipeline_cache: Rework to use multialloc and blob\n\nThis gets rid of all of our hand-rolled size calculation and\nserialization code and replaces it with safe "standards"