On 2013/01/02 19:14:20, apatrick1 wrote: > LGTM. Do you need me to commit it? C4267 ...
11 years, 10 months ago
(2013-01-02 19:21:22 UTC)
#4
On 2013/01/02 19:14:20, apatrick1 wrote:
> LGTM. Do you need me to commit it?
C4267 is actually quite a useful warning. Ideally the code should be corrected
to have no implicit 64-bit to 32-bit conversions.
On 2013/01/02 19:21:22, nicolas wrote: > On 2013/01/02 19:14:20, apatrick1 wrote: > > LGTM. Do ...
11 years, 10 months ago
(2013-01-02 19:22:45 UTC)
#5
On 2013/01/02 19:21:22, nicolas wrote:
> On 2013/01/02 19:14:20, apatrick1 wrote:
> > LGTM. Do you need me to commit it?
>
> C4267 is actually quite a useful warning. Ideally the code should be corrected
> to have no implicit 64-bit to 32-bit conversions.
Ideally it should yes, but right now we have warnings-as-errors turned on, so it
fails to build.
On 2013/01/02 19:21:22, nicolas wrote: > On 2013/01/02 19:14:20, apatrick1 wrote: > > LGTM. Do ...
11 years, 10 months ago
(2013-01-02 19:23:19 UTC)
#6
On 2013/01/02 19:21:22, nicolas wrote:
> On 2013/01/02 19:14:20, apatrick1 wrote:
> > LGTM. Do you need me to commit it?
>
> C4267 is actually quite a useful warning. Ideally the code should be corrected
> to have no implicit 64-bit to 32-bit conversions.
Yes, the plan is not to keep the disabled warnings permanent. But in the interim
I'm disabling them on a per package basis to get a Win64 build stood up.
Issue 7011048: Make angle compile on Win64
(Closed)
Created 11 years, 11 months ago by jschuh
Modified 11 years, 10 months ago
Reviewers: apatrick1, nicolas, dgkoch
Base URL: http://angleproject.googlecode.com/svn/trunk/
Comments: 0