Fixed Display::initialize so it does not crash for devices that do not suppor...

Fixed Display::initialize so it does not crash for devices that do not support PS 2.0.

The crash occurs when createDevice attempts to dereference mD3d9, which is NULL.

[apatrick1, 2010-08-11 22:05:17]

Hi,

This is a fairly simple change but it's pretty high priority for us because it's
making the servers on our build farm randomly crash on account of not supporting
PS 2.0. I know Dan is on vacation. Is there anyone else who could review this in
the meantime?

Thanks,

Al

[dgkoch, 2010-08-11 23:48:13]

Looks alright.  I believe the bulk of the changes is just indentation, correct?

[apatrick1, 2010-08-11 23:59:35]

Hi Dan, Thanks for the fast feedback. You are correct about the indentation.

Al

http://codereview.appspot.com/1956042/diff/1/2
File src/libEGL/Display.cpp (right):

http://codereview.appspot.com/1956042/diff/1/2#newcode74
src/libEGL/Display.cpp:74: return error(EGL_NOT_INITIALIZED, false);
The only change functionality is this block here that earlies out if the PS
model is less than 2.0.

http://codereview.appspot.com/1956042/diff/1/2#newcode76
src/libEGL/Display.cpp:76: 
Changes from here down are just indentation changes to account from the removal
of the else branch.