Scale function cleaned up white spaces and removed comments that are no longe...

Scale function cleaned up white spaces and removed comments that are no longer applicable to this module.
Removed openmp support, which is no longer necessary for high performance.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=19971

[fbarchard, 2009-06-30 23:56:55]

cleaning up code/comments

[scherkus (not reviewing), 2009-07-02 00:25:30]

tiny nits on commenting

http://codereview.chromium.org/150143/diff/1/2
File media/base/yuv_convert.cc (right):

http://codereview.chromium.org/150143/diff/1/2#newcode23
Line 23: namespace media {
nit: add blank line after namespace

http://codereview.chromium.org/150143/diff/1/2#newcode48
Line 48: EMMS();
nit: add a one-line comment explaining why we're executing EMMS

http://codereview.chromium.org/150143/diff/1/2#newcode159
Line 159: EMMS();
nit: add a one-line comment explaining why we're executing EMMS

[fbarchard, 2009-07-06 07:41:39]

Done.

http://codereview.chromium.org/150143/diff/1/2
File media/base/yuv_convert.cc (right):

http://codereview.chromium.org/150143/diff/1/2#newcode23
Line 23: namespace media {
On 2009/07/02 00:25:30, scherkus wrote:
> nit: add blank line after namespace

Done.

http://codereview.chromium.org/150143/diff/1/2#newcode48
Line 48: EMMS();
On 2009/07/02 00:25:30, scherkus wrote:
> nit: add a one-line comment explaining why we're executing EMMS

Done.

http://codereview.chromium.org/150143/diff/1/2#newcode159
Line 159: EMMS();
On 2009/07/02 00:25:30, scherkus wrote:
> nit: add a one-line comment explaining why we're executing EMMS

Done.

[scherkus (not reviewing), 2009-07-06 19:47:21]

tiny nits, but LGTM

http://codereview.chromium.org/150143/diff/2001/3001
File media/base/yuv_convert.cc (right):

http://codereview.chromium.org/150143/diff/2001/3001#newcode49
Line 49: // MMX used for FastConvertYUVToRGB32Row requires emms instruction.
nit: blank line before comments, unless it's first line in new scope block

http://codereview.chromium.org/150143/diff/2001/3001#newcode161
Line 161: // MMX used for FastConvertYUVToRGB32Row requires emms instruction.
nit: blank line before comments, unless it's first line in new scope block