Commita61d95f

Clement Champetier

committed

VideoProperties: refactor how to getDuration

Else statement was not necessary!

1 parentad44be7 commita61d95fCopy full SHA for a61d95f

File tree

-10

lines changed

-10

lines changed

Lines changed: 8 additions & 10 deletions

Original file line number	Diff line number	Diff line change
`@@ -475,18 +475,16 @@ float VideoProperties::getDuration() const`
`475`	`475`	`LOG_INFO("Need a deeper analysis: see eAnalyseLevelFirstGop.")`
`476`	`476`	`return0;`
`477`	`477`	`}`
`478`		`-else`
	`478`	`+`
	`479`	`+if(! _nbFrames)`
`479`	`480`	`{`
`480`		`-if(! _nbFrames)`
`481`		`- {`
`482`		`-LOG_INFO("Estimate the duration from the file size and the bitrate.")`
`483`		`-constsize_t bitRate =getBitRate();`
`484`		`-if(bitRate)`
`485`		`-return _fileProperties->getFileSize() / bitRate *8;`
`486`		`- }`
`487`		`-LOG_INFO("Get the exact duration from the number of frames and the fps.")`
`488`		`-return _nbFrames /getFps();`
	`481`	`+LOG_INFO("Estimate the duration from the file size and the bitrate.")`
	`482`	`+constsize_t bitRate =getBitRate();`
	`483`	`+if(bitRate)`
	`484`	`+return _fileProperties->getFileSize() / bitRate *8;`
`489`	`485`	`}`
	`486`	`+LOG_INFO("Get the exact duration from the number of frames and the fps.")`
	`487`	`+return _nbFrames /getFps();`
`490`	`488`	`}`
`491`	`489`
`492`	`490`	`boolVideoProperties::hasBFrames()const`

Comments

(0)