Opened 3 years ago
Closed 3 years ago
#1489 closed defect (fixed)
Cleanup of variable names for m_iSourceWidth and m_iSourceHeight
Reported by: | wangyang.cs | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | VTM-13.1 |
Component: | VTM | Version: | VTM-12.0 |
Keywords: | Cc: | ksuehring, bbross, XiangLi, fbossen, jvet@…, jvet@… |
Description
m_iSourceWidth and m_iSourceHeight were changed to m_sourceWidth and m_sourceHeight from VTM-12.0 to VTM-13.0 for cleanups. In the latest VTM, Several remaining ones need to be renamed.
#if EXTENSION_360_VIDEO
m_inputFileWidth = m_iSourceWidth;
m_inputFileHeight = m_iSourceHeight;
m_ext360.setMaxCUInfo(m_uiCTUSize, 1 << MIN_CU_LOG2);
#endif
#if QP_SWITCHING_FOR_PARALLEL
if ((m_iQP < 38) && m_bUsePerceptQPA && !m_bUseAdaptiveQP && (m_sourceWidth <= 2048) && (m_sourceHeight <= 1280)
#else
if (((int)m_fQP < 38) && m_bUsePerceptQPA && !m_bUseAdaptiveQP && (m_iSourceWidth <= 2048) && (m_iSourceHeight <= 1280)
#endif
#if QP_SWITCHING_FOR_PARALLEL
if( ( m_iQP < 38 ) && ( m_iGOPSize > 4 ) && m_bUsePerceptQPA && !m_bUseAdaptiveQP && ( m_iSourceHeight <= 1280 ) && ( m_iSourceWidth <= 2048 ) )
#else
if( ( ( int ) m_fQP < 38 ) && ( m_iGOPSize > 4 ) && m_bUsePerceptQPA && !m_bUseAdaptiveQP && ( m_iSourceHeight <= 1280 ) && ( m_iSourceWidth <= 2048 ) )
#endif
Change history (2)
comment:1 Changed 3 years ago by wangyang.cs
comment:2 Changed 3 years ago by XiangLi
- Milestone set to VTM-13.1
- Resolution set to fixed
- Status changed from new to closed
Fixed as suggested.
A MR https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/-/merge_requests/2082 is submitted.