aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndreas Gampe <agampe@google.com>2014-12-08 19:41:55 -0800
committerAndreas Gampe <agampe@google.com>2014-12-08 19:43:28 -0800
commit1edf1d805b1d18cc014af280f00b8de346276dd0 (patch)
treec1dc64cca204c4533d5f4cc8242bb4af8bc5f7ee
parent8a050ed9dc33830ebab505940e05e36a1a999080 (diff)
downloadandroid_hardware_intel_common_omx-components-1edf1d805b1d18cc014af280f00b8de346276dd0.tar.gz
android_hardware_intel_common_omx-components-1edf1d805b1d18cc014af280f00b8de346276dd0.tar.bz2
android_hardware_intel_common_omx-components-1edf1d805b1d18cc014af280f00b8de346276dd0.zip
VideoCodec: Remove unused variables
For build-system CFLAGS clean-up, remove unused variables. Bug: 18632512 Change-Id: I7644f8bc2355c27b199ec69097774b579e836048
-rw-r--r--videocodec/OMXVideoDecoderBase.h2
-rw-r--r--videocodec/OMXVideoDecoderVP9HWR.cpp8
-rw-r--r--videocodec/OMXVideoDecoderVP9Hybrid.cpp11
-rw-r--r--videocodec/OMXVideoEncoderBase.cpp8
-rwxr-xr-xvideocodec/securevideo/moorefield/OMXVideoDecoderAVCSecure.cpp2
5 files changed, 1 insertions, 30 deletions
diff --git a/videocodec/OMXVideoDecoderBase.h b/videocodec/OMXVideoDecoderBase.h
index 556bc23..4f836a4 100644
--- a/videocodec/OMXVideoDecoderBase.h
+++ b/videocodec/OMXVideoDecoderBase.h
@@ -27,7 +27,7 @@
#include "graphics.h"
#endif
-static const char* VA_VED_RAW_MIME_TYPE = "video/x-raw-vaved";
+static constexpr const char* VA_VED_RAW_MIME_TYPE = "video/x-raw-vaved";
static const uint32_t VA_VED_COLOR_FORMAT = 0x20;
diff --git a/videocodec/OMXVideoDecoderVP9HWR.cpp b/videocodec/OMXVideoDecoderVP9HWR.cpp
index ec97bac..df64314 100644
--- a/videocodec/OMXVideoDecoderVP9HWR.cpp
+++ b/videocodec/OMXVideoDecoderVP9HWR.cpp
@@ -90,8 +90,6 @@ OMXVideoDecoderVP9HWR::~OMXVideoDecoderVP9HWR()
{
LOGV("OMXVideoDecoderVP9HWR is destructed.");
- unsigned int i = 0;
-
if (mVADisplay) {
vaTerminate(mVADisplay);
mVADisplay = NULL;
@@ -411,7 +409,6 @@ OMX_ERRORTYPE OMXVideoDecoderVP9HWR::ProcessorProcess(
{
OMX_ERRORTYPE ret;
OMX_BUFFERHEADERTYPE *inBuffer = *pBuffers[INPORT_INDEX];
- OMX_BUFFERHEADERTYPE *outBuffer = *pBuffers[OUTPORT_INDEX];
OMX_BOOL isResolutionChange = OMX_FALSE;
if (inBuffer->pBuffer == NULL) {
@@ -603,11 +600,6 @@ OMX_ERRORTYPE OMXVideoDecoderVP9HWR::FillRenderBuffer(OMX_BUFFERHEADERTYPE **pBu
void *dst = buffer->pBuffer;
uint8_t *dst_y = (uint8_t *)dst;
- const OMX_PARAM_PORTDEFINITIONTYPE *paramPortDefinitionInput
- = this->ports[INPORT_INDEX]->GetPortDefinition();
-
- size_t inBufferWidth = paramPortDefinitionInput->format.video.nFrameWidth;
- size_t inBufferHeight = paramPortDefinitionInput->format.video.nFrameHeight;
const OMX_PARAM_PORTDEFINITIONTYPE *paramPortDefinitionOutput
= this->ports[OUTPORT_INDEX]->GetPortDefinition();
diff --git a/videocodec/OMXVideoDecoderVP9Hybrid.cpp b/videocodec/OMXVideoDecoderVP9Hybrid.cpp
index e1ff00d..d7f42fd 100644
--- a/videocodec/OMXVideoDecoderVP9Hybrid.cpp
+++ b/videocodec/OMXVideoDecoderVP9Hybrid.cpp
@@ -118,7 +118,6 @@ OMX_ERRORTYPE OMXVideoDecoderVP9Hybrid::ProcessorFlush(OMX_U32) {
OMX_ERRORTYPE OMXVideoDecoderVP9Hybrid::ProcessorPreFillBuffer(OMX_BUFFERHEADERTYPE* buffer) {
unsigned int handle = (unsigned int)buffer->pBuffer;
- unsigned int i = 0;
if (buffer->nOutputPortIndex == OUTPORT_INDEX){
mSingalRenderDone(handle);
@@ -133,7 +132,6 @@ OMX_ERRORTYPE OMXVideoDecoderVP9Hybrid::ProcessorProcess(
{
OMX_ERRORTYPE ret;
OMX_BUFFERHEADERTYPE *inBuffer = *pBuffers[INPORT_INDEX];
- OMX_BUFFERHEADERTYPE *outBuffer = *pBuffers[OUTPORT_INDEX];
if (inBuffer->pBuffer == NULL) {
LOGE("Buffer to decode is empty.");
@@ -199,11 +197,6 @@ OMX_ERRORTYPE OMXVideoDecoderVP9Hybrid::ProcessorProcess(
return ret;
}
-static int ALIGN(int x, int y) {
- // y must be a power of 2.
- return (x + y - 1) & ~(y - 1);
-}
-
OMX_ERRORTYPE OMXVideoDecoderVP9Hybrid::FillRenderBuffer(OMX_BUFFERHEADERTYPE **pBuffer,
buffer_retain_t *retain,
OMX_U32 inportBufferFlags)
@@ -225,10 +218,6 @@ OMX_ERRORTYPE OMXVideoDecoderVP9Hybrid::FillRenderBuffer(OMX_BUFFERHEADERTYPE **
buffer = *pBuffer = mOMXBufferHeaderTypePtrArray[fb_index];
- size_t dst_y_size = mGraphicBufferParam.graphicBufferStride *
- mGraphicBufferParam.graphicBufferHeight;
- size_t dst_c_stride = ALIGN(mGraphicBufferParam.graphicBufferStride / 2, 16);
- size_t dst_c_size = dst_c_stride * mGraphicBufferParam.graphicBufferHeight / 2;
buffer->nOffset = 0;
buffer->nFilledLen = sizeof(OMX_U8*);
if (inportBufferFlags & OMX_BUFFERFLAG_EOS) {
diff --git a/videocodec/OMXVideoEncoderBase.cpp b/videocodec/OMXVideoEncoderBase.cpp
index bc37f4f..9438e8c 100644
--- a/videocodec/OMXVideoEncoderBase.cpp
+++ b/videocodec/OMXVideoEncoderBase.cpp
@@ -267,17 +267,13 @@ OMX_ERRORTYPE OMXVideoEncoderBase::SetVideoEncoderParam() {
Encode_Status ret = ENCODE_SUCCESS;
PortVideo *port_in = NULL;
- PortVideo *port_out = NULL;
- OMX_VIDEO_CONTROLRATETYPE controlrate;
const OMX_PARAM_PORTDEFINITIONTYPE *paramPortDefinitionInput = NULL;
LOGV("OMXVideoEncoderBase::SetVideoEncoderParam called\n");
port_in = static_cast<PortVideo *>(ports[INPORT_INDEX]);
- port_out = static_cast<PortVideo *>(ports[OUTPORT_INDEX]);
paramPortDefinitionInput = port_in->GetPortDefinition();
mEncoderParams->resolution.height = paramPortDefinitionInput->format.video.nFrameHeight;
mEncoderParams->resolution.width = paramPortDefinitionInput->format.video.nFrameWidth;
- const OMX_VIDEO_PARAM_BITRATETYPE *bitrate = port_out->GetPortBitrateParam();
mEncoderParams->frameRate.frameRateDenom = 1;
if(mConfigFramerate.xEncodeFramerate != 0) {
@@ -356,8 +352,6 @@ OMX_ERRORTYPE OMXVideoEncoderBase::ProcessorInit(void) {
}
OMX_ERRORTYPE OMXVideoEncoderBase::ProcessorDeinit(void) {
- OMX_ERRORTYPE ret;
-
if(mVideoEncoder) {
mVideoEncoder->stop();
}
@@ -806,8 +800,6 @@ OMX_ERRORTYPE OMXVideoEncoderBase::SetStoreMetaDataInBuffers(OMX_PTR pStructure)
StoreMetaDataInBuffersParams *p = (StoreMetaDataInBuffersParams *)pStructure;
VideoParamsStoreMetaDataInBuffers StoreMetaDataInBuffers;
PortVideo *port = static_cast<PortVideo *>(this->ports[INPORT_INDEX]);
- PortVideo *output_port = static_cast<PortVideo *>(this->ports[OUTPORT_INDEX]);
- uint32_t maxSize = 0;
CHECK_TYPE_HEADER(p);
CHECK_PORT_INDEX(p, INPORT_INDEX);
diff --git a/videocodec/securevideo/moorefield/OMXVideoDecoderAVCSecure.cpp b/videocodec/securevideo/moorefield/OMXVideoDecoderAVCSecure.cpp
index 1cee46c..101a34d 100755
--- a/videocodec/securevideo/moorefield/OMXVideoDecoderAVCSecure.cpp
+++ b/videocodec/securevideo/moorefield/OMXVideoDecoderAVCSecure.cpp
@@ -147,8 +147,6 @@ OMX_ERRORTYPE OMXVideoDecoderAVCSecure::ProcessorDeinit(void) {
}
OMX_ERRORTYPE OMXVideoDecoderAVCSecure::ProcessorStart(void) {
- uint32_t imrOffset = 0;
- uint32_t dataBufferSize = DATA_BUFFER_SIZE;
EnableIEDSession(true);
uint32_t ret = drm_vendor_api.drm_start_playback();