V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
yfqh
V2EX  ›  程序员

C++代码问题咨询

  •  
  •   yfqh · 15 天前 · 690 次点击
    我在修改 Chromium 的代码,在修改到下面这段的时候,发现代码存在问题,出现问题的地方是在(has_poster_xxx)这里,但我暂时没想到解决的办法,请问大家知道怎样修改这里的代码吗?

    auto start_type = media::Pipeline::StartType::kNormal;
    if ((GetDemuxerType() != media::DemuxerType::kChunkDemuxer) &&
    preload_ == media::DataSource::METADATA &&
    !client_->CouldPlayIfEnoughData() && !IsStreaming()) {
    start_type =
    (has_poster_
    ? media::Pipeline::StartType::kSuspendAfterMetadata
    : media::Pipeline::StartType::kSuspendAfterMetadataForAudioOnly);
    attempting_suspended_start_ = true;
    }
    2 条回复    2022-11-24 20:15:58 +08:00
    brucedone
        1
    brucedone  
       15 天前
    出的啥问题呢?没太懂你想改啥,比如期望的输入和输出是什么
    yfqh
        2
    yfqh  
    OP
       15 天前
    @brucedone 下面是初始的代码段

    auto start_type = media::Pipeline::StartType::kNormal;
    if ((GetDemuxerType() != media::DemuxerType::kChunkDemuxer) &&
    preload_ == media::DataSource::METADATA &&
    !client_->CouldPlayIfEnoughData() && !IsStreaming()) {
    start_type =
    (has_poster_ ||
    base::FeatureList::IsEnabled(media::kPreloadMetadataLazyLoad))
    ? media::Pipeline::StartType::kSuspendAfterMetadata
    : media::Pipeline::StartType::kSuspendAfterMetadataForAudioOnly;
    attempting_suspended_start_ = true;
    }

    这是我修改后的

    auto start_type = media::Pipeline::StartType::kNormal;
    if ((GetDemuxerType() != media::DemuxerType::kChunkDemuxer) &&
    preload_ == media::DataSource::METADATA &&
    !client_->CouldPlayIfEnoughData() && !IsStreaming()) {
    start_type =
    (has_poster_
    ? media::Pipeline::StartType::kSuspendAfterMetadata
    : media::Pipeline::StartType::kSuspendAfterMetadataForAudioOnly);
    attempting_suspended_start_ = true;
    }

    我现在要将 base::FeatureList::IsEnabled(media::kPreloadMetadataLazyLoad)删去,但删去后好像发现会存在其他问题,这是 Chromium 工程师给的回复,我暂时还没有其他的修改思路。

    The original code was: start_type = (has_poster_ || true) ? kSuspendAfterMetadata : kSuspendAfterMetadataForAudioOnly;

    Patchset 9 changed it to: start_type = (has_poster_) ? kSuspendAfterMetadata : kSuspendAfterMetadataForAudioOnly;

    Patchset 11 changed it to: start_type = (has_poster_ ? kSuspendAfterMetadata : kSuspendAfterMetadataForAudioOnly);

    Shouldn't the code be: start_type = kSuspendAfterMetadata; ?

    There may be some additional cleanup of kSuspendAfterMetadataForAudioOnly.
    关于   ·   帮助文档   ·   API   ·   FAQ   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   2846 人在线   最高记录 5497   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 38ms · UTC 12:42 · PVG 20:42 · LAX 04:42 · JFK 07:42
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.