Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1a0542f9 authored by Greg Kaiser's avatar Greg Kaiser
Browse files

Codec2Mapper: Properly lower case strings 

We passed arguments in the wrong order to std::transform() in
a couple places, resulting in our strings not getting properly
lower cased.  We fix the argument order.

Test: TreeHugger
Change-Id: Ie6eda30de0e6a07130a75967fd2a0e131d1cf3b1
parent 3ddf15a0

media/codec2/sfplugin/utils/Codec2Mapper.cpp

+2 −2
Original line number Diff line number Diff line
@@ -629,7 +629,7 @@ private: 
// static

std::shared_ptr<C2Mapper::ProfileLevelMapper>

C2Mapper::GetProfileLevelMapper(std::string mediaType) {

    std::transform(mediaType.begin(), mediaType.begin(), mediaType.end(), ::tolower);

    std::transform(mediaType.begin(), mediaType.end(), mediaType.begin(), ::tolower);

    if (mediaType == MIMETYPE_AUDIO_AAC) {

        return std::make_shared<AacProfileLevelMapper>();

    } else if (mediaType == MIMETYPE_VIDEO_AVC) {
@@ -657,7 +657,7 @@ C2Mapper::GetProfileLevelMapper(std::string mediaType) { 
// static

std::shared_ptr<C2Mapper::ProfileLevelMapper>

C2Mapper::GetHdrProfileLevelMapper(std::string mediaType, bool isHdr10Plus) {

    std::transform(mediaType.begin(), mediaType.begin(), mediaType.end(), ::tolower);

    std::transform(mediaType.begin(), mediaType.end(), mediaType.begin(), ::tolower);

    if (mediaType == MIMETYPE_VIDEO_HEVC) {

        return std::make_shared<HevcProfileLevelMapper>(true, isHdr10Plus);

    } else if (mediaType == MIMETYPE_VIDEO_VP9) {