[mythtv-commits] mythtv branch master updated by danielk. v0.27-pre2-352-g5a8b3ed
Git Repo Owner
noreply at mythtv.org
Thu Dec 13 14:12:43 UTC 2012
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(-)
--
More information about the mythtv-commits
mailing list