[mythtv-firehose] mythtv/master commit: 9f85e4efd by Gavin Hurlbut (Beirdo)

MythTV noreply at mythtv.org
Wed Oct 19 18:59:39 UTC 2011

      Author:  Gavin Hurlbut <ghurlbut at mythtv.org>
 Change Date:  2011-10-19T11:52:43-07:00
   Push Date:  2011/10/19 11:59:20 -0700
  Repository:  mythtv
      Branch:  master
New Revision:  9f85e4efd24f44e5d9dd8bbf9f0c3a5ce718c92d
   Changeset:  https://github.com/MythTV/mythtv/commit/9f85e4efd


Attempt to reduce stupid warnings on -ppc buildslave

The linux-ppc buildslave is using a newer version of gcc than the others,
so it has pointed out some extra warnings.  The warning that this should
permanently remove is:

./../external/FFmpeg/libavcodec/get_bits.h:305:45: warning: variable ‘re_cache’ set but not used [-Wunused-but-set-variable]

This is caused when using the skip_bits() function as the OPEN_READER() macro
creates a re_cache variable, but nothing uses it in that specific call flow.
The fix is to specifically reference it in the CLOSE_READER() macro so it is
always used.  A void cast reference should be enough to make gcc happy, but
yet not affect the output code.

This has been tested on the older build systems, but not yet on the ppc slave.
If it goes awry, I'll revert it.  We should look at pushing this upstream (to
one or both upstreams) if it proves to work well.



More information about the mythtv-firehose mailing list