24 Oct
2015
24 Oct
'15
9:38 p.m.
On Sat, 24 Oct 2015, Gerald Pfeifer wrote:
This avoids potential shift overflows. I only should
be necessary
for D3DVSD_TOKENTYPESHIFT, but it makes sense to have this consistent.
Correction, this per se does not avoid shift overflows, I sent an
older patch. It still may make sense to apply (please advise), but
we'd need a different change in addition. I'll have a look.
Gerald
26 Oct
26 Oct
8:31 a.m.
New subject: d3d8types.h: Convert the D3DVSD_...SHIFT constants to unsigned.
On 24 October 2015 at 23:38, Gerald Pfeifer <gerald(a)pfeifer.com> wrote:
On Sat, 24 Oct 2015, Gerald Pfeifer wrote:
I don't
think it really makes sense to make the shifts unsigned. There
might be an argument for consistently using unsigned constants unless
they need to be signed, but that would be a much larger change.
This avoids potential shift overflows. I only
should be necessary
for D3DVSD_TOKENTYPESHIFT, but it makes sense to have this consistent.
Correction, this per se does not avoid shift overflows, I sent an
older patch. It still may make sense to apply (please advise), but
30 Oct
30 Oct
8:07 a.m.
New subject: d3d8types.h: Convert the D3DVSD_...SHIFT constants to unsigned.
On Mon, 26 Oct 2015, Henri Verbeet wrote:
I don't think it really makes sense to make the
shifts unsigned. There
might be an argument for consistently using unsigned constants unless
they need to be signed, but that would be a much larger change.
Okay. If you want I could have a look, say converting one header
file, but I agree, it's not much of a priority. Let me know in
case.
Gerald
3120
days inactive
3126
days old
2 comments
2 participants
participants (2)
-
Gerald Pfeifer -
Henri Verbeet