[mythtv-firehose] mythtv branch master updated by danielk. v0.27-pre2-352-g5a8b3ed

[Git Repo Owner]

The branch, master has been updated on the
mythtv repository by gitolite user danielk.
       via  5a8b3ed5d80e58d21dd747171dfc3c6289198362 (commit)
      from  7c7b44a7da27add500990307144a1e477dcfaff6 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 5a8b3ed5d80e58d21dd747171dfc3c6289198362
Author:    Daniel Thor Kristjansson <danielk at cuymedia.net> at Thu, 13 Dec 2012 08:54:06 -0500
Committer: Daniel Thor Kristjansson <danielk at cuymedia.net> at Thu, 13 Dec 2012 09:12:35 -0500
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=5a8b3ed5d80e58d21dd747171dfc3c6289198362

Rearrange mmx_t union to make gcc happy.
When we initialize with an unsigned long long using:
  mmx_t var = { 0x8000000000000000ULL };
It makes the compiler unhappy because it tries to
assign the mmx_t's .q which is of type long long
and this number is too large to be represented in
a long long. By moving the unsigned long long .uq to
be first in the union the compiler will try that
first and not give a narrowing complaint.
An alternative in C99 code is:
  mmx_t var = { .uq = 0x8000000000000000ULL };
But in C++ the only alternative is:
  mmx_t var;
  var.uq = 0x8000000000000000ULL;
Which to my eyes is less elegant.



-----------------------------------------------------------------------

Summary of changes:
 mythtv/libs/libmythbase/ffmpeg-mmx.h |    7 ++++++-
 1 files changed, 6 insertions(+), 1 deletions(-)

--