actions/itgmania212121
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

SaveScreenshot: remove unused timer

Browse Source 
The timer is only used by the commented-out debug log methods, but the timer itself still runs. This removes the timer as well as the unused logs.
This commit is contained in:
sukibaby
2025-04-29 23:25:27 -07:00
committed by teejusb
parent b5a2f86277
commit a3eb3d8646
1 changed files with 1 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/RageDisplay.cpp
+1 -7
View File
@@ -769,10 +769,8 @@ void RageDisplay::UpdateCentering()
bool RageDisplay::SaveScreenshot( const RString &sPath, GraphicsFileFormat format )
{
RageTimer timer;
RageSurface *surface = this->CreateScreenshot();
// LOG->Trace( "CreateScreenshot took %f seconds", timer.GetDeltaTime() );
if (nullptr == surface)
{
LOG->Warn("SaveScreenshot: failed to create a screenshot surface");
@@ -790,9 +788,7 @@ bool RageDisplay::SaveScreenshot( const RString &sPath, GraphicsFileFormat forma
// This used to be lrint. However, lrint causes odd resolutions like
// 639x480 (4:3) and 853x480 (16:9). ceil gives correct values. -aj
int iWidth = std::ceil( iHeight * GetActualVideoModeParams().fDisplayAspectRatio );
timer.Touch();
RageSurfaceUtils::Zoom( surface, iWidth, iHeight );
// LOG->Trace( "%ix%i -> %ix%i (%.3f) in %f seconds", surface->w, surface->h, iWidth, iHeight, GetActualVideoModeParams().fDisplayAspectRatio, timer.GetDeltaTime() );
}
RageFile out;
@@ -804,7 +800,6 @@ bool RageDisplay::SaveScreenshot( const RString &sPath, GraphicsFileFormat forma
}
bool bSuccess = false;
timer.Touch();
RString strError = "";
switch( format )
{
@@ -825,7 +820,6 @@ bool RageDisplay::SaveScreenshot( const RString &sPath, GraphicsFileFormat forma
bSuccess = false;
break;
}
// LOG->Trace( "Saving Screenshot file took %f seconds.", timer.GetDeltaTime() );
RageUtil::SafeDelete( surface );