Cleanup, adjustscanlines is now vsizeadjust.

This commit is contained in:
Christian Speckner 2020-01-13 20:27:05 +01:00
parent bb14b775b6
commit cad4c627f5
9 changed files with 33 additions and 38 deletions

View File

@ -685,25 +685,22 @@ void Console::updateVcenter(Int32 vcenter)
// - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
void Console::changeScanlineAdjust(int direction) void Console::changeScanlineAdjust(int direction)
{ {
// Get correct setting, depending on whether we're in NTSC or PAL float newVsizeAdjust = myTIA->vsizeAdjust();
// Int32 adjust = myOSystem.settings().getInt("tia.adjustscanlines.ntsc");
float newAdjustScanliens = myTIA->adjustScanlines();;
if (direction != -1 && direction != +1) return; if (direction != -1 && direction != +1) return;
if(direction == +1) // increase scanline adjustment if(direction == +1) // increase scanline adjustment
{ {
newAdjustScanliens = std::min(myTIA->adjustScanlines() + 0.5f, 5.f); newVsizeAdjust = std::min(myTIA->vsizeAdjust() + 0.5f, 5.f);
} }
else if(direction == -1) // decrease scanline adjustment else if(direction == -1) // decrease scanline adjustment
{ {
newAdjustScanliens = std::max(myTIA->adjustScanlines() - 0.5f, -5.f); newVsizeAdjust = std::max(myTIA->vsizeAdjust() - 0.5f, -5.f);
} }
if (newAdjustScanliens != myTIA->adjustScanlines()) { if (newVsizeAdjust != myTIA->vsizeAdjust()) {
myTIA->setAdjustScanlines(newAdjustScanliens); myTIA->setVsizeAdjust(newVsizeAdjust);
myOSystem.settings().setValue("tia.adjustscanlines", newAdjustScanliens); myOSystem.settings().setValue("tia.vsizeadjust", newVsizeAdjust);
initializeVideo(); initializeVideo();
} }
} }
@ -727,7 +724,7 @@ void Console::setTIAProperties()
myTIA->setLayout(FrameLayout::pal); myTIA->setLayout(FrameLayout::pal);
} }
myTIA->setAdjustScanlines(myOSystem.settings().getFloat("tia.adjustscanlines")); myTIA->setVsizeAdjust(myOSystem.settings().getFloat("tia.vsizeadjust"));
myTIA->setVcenter(vcenter); myTIA->setVcenter(vcenter);
myEmulationTiming.updateFrameLayout(myTIA->frameLayout()); myEmulationTiming.updateFrameLayout(myTIA->frameLayout());

View File

@ -50,7 +50,7 @@ Settings::Settings()
// TIA specific options // TIA specific options
setPermanent("tia.zoom", "3"); setPermanent("tia.zoom", "3");
setPermanent("tia.inter", "false"); setPermanent("tia.inter", "false");
setPermanent("tia.adjustscanlines", 0.f); setPermanent("tia.vsizeadjust", 0.f);
setPermanent("fullscreen", "false"); setPermanent("fullscreen", "false");
setPermanent("tia.fs_stretch", "false"); setPermanent("tia.fs_stretch", "false");
setPermanent("tia.fs_overscan", "0"); setPermanent("tia.fs_overscan", "0");
@ -254,8 +254,8 @@ void Settings::validate()
f = getFloat("speed"); f = getFloat("speed");
if (f <= 0) setValue("speed", "1.0"); if (f <= 0) setValue("speed", "1.0");
i = getFloat("tia.adjustscanlines"); i = getFloat("tia.vsizeadjust");
if(i < -5. || i > 5.) setValue("tia.adjustscanlines", 0.f); if(i < -5. || i > 5.) setValue("tia.vsizeadjust", 0.f);
s = getString("tia.dbgcolors"); s = getString("tia.dbgcolors");
sort(s.begin(), s.end()); sort(s.begin(), s.end());
@ -407,7 +407,7 @@ void Settings::usage() const
<< " for TIA image\n" << " for TIA image\n"
<< " -tia.inter <1|0> Enable interpolated (smooth) scaling for TIA\n" << " -tia.inter <1|0> Enable interpolated (smooth) scaling for TIA\n"
<< " image\n" << " image\n"
<< " -tia.adjustscanlines <float> Adjust the visible number if TIA scanlines [percent]\n" << " -tia.vsizeadjust <float> Adjust the vertical range of the image [percent]\n"
<< " -tia.fs_stretch <1|0> Stretch TIA image to fill fullscreen mode\n" << " -tia.fs_stretch <1|0> Stretch TIA image to fill fullscreen mode\n"
<< " -tia.fs_overscan <0-10> Add overscan to TIA image in fill fullscreen mode\n" << " -tia.fs_overscan <0-10> Add overscan to TIA image in fill fullscreen mode\n"
<< " -tia.dbgcolors <string> Debug colors to use for each object (see manual\n" << " -tia.dbgcolors <string> Debug colors to use for each object (see manual\n"

View File

@ -264,8 +264,8 @@ class TIA : public Device
void setLayout(FrameLayout layout) { myFrameManager->setLayout(layout); } void setLayout(FrameLayout layout) { myFrameManager->setLayout(layout); }
FrameLayout frameLayout() const { return myFrameManager->layout(); } FrameLayout frameLayout() const { return myFrameManager->layout(); }
void setAdjustScanlines(float adjustScanlines) { myFrameManager->setAdjustScanlines(adjustScanlines); } void setVsizeAdjust(float vsizeAdjust) { myFrameManager->setVsizeAdjust(vsizeAdjust); }
float adjustScanlines() const { return myFrameManager->adjustScanlines(); } float vsizeAdjust() const { return myFrameManager->vsizeAdjust(); }
/** /**
Enables/disables color-loss for PAL modes only. Enables/disables color-loss for PAL modes only.

View File

@ -181,9 +181,9 @@ class AbstractFrameManager : public Serializable
virtual Int32 maxVcenter() const { return 0; } virtual Int32 maxVcenter() const { return 0; }
virtual void setAdjustScanlines(float adjustScanlines) {} virtual void setVsizeAdjust(float vsizeAdjust) {}
virtual float adjustScanlines() const { return 0; } virtual float vsizeAdjust() const { return 0; }
/** /**
* The corresponding start line. * The corresponding start line.

View File

@ -121,9 +121,9 @@ void FrameManager::setVcenter(Int32 vcenter)
} }
// - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
void FrameManager::setAdjustScanlines(float adjustScanlines) void FrameManager::setVsizeAdjust(float vsizeAdjust)
{ {
myAdjustScanlines = adjustScanlines; myVsizeAdjust = vsizeAdjust;
recalculateMetrics(); recalculateMetrics();
} }
@ -244,12 +244,10 @@ void FrameManager::recalculateMetrics() {
throw runtime_error("frame manager: invalid TV mode"); throw runtime_error("frame manager: invalid TV mode");
} }
myHeight = BSPF::clamp<uInt32>(round(static_cast<float>(baseHeight) * (1.f + myAdjustScanlines / 100.f)), 0, myFrameLines); myHeight = BSPF::clamp<uInt32>(round(static_cast<float>(baseHeight) * (1.f + myVsizeAdjust / 100.f)), 0, myFrameLines);
myYStart = BSPF::clamp<uInt32>(ystartBase + (baseHeight - static_cast<Int32>(myHeight)) / 2 - myVcenter, 0, myFrameLines); myYStart = BSPF::clamp<uInt32>(ystartBase + (baseHeight - static_cast<Int32>(myHeight)) / 2 - myVcenter, 0, myFrameLines);
// TODO: why "- 1" here: ??? // TODO: why "- 1" here: ???
myMaxVcenter = BSPF::clamp<Int32>(ystartBase + (baseHeight - static_cast<Int32>(myHeight)) / 2 - 1, 0, TIAConstants::maxVcenter); myMaxVcenter = BSPF::clamp<Int32>(ystartBase + (baseHeight - static_cast<Int32>(myHeight)) / 2 - 1, 0, TIAConstants::maxVcenter);
cout << myAdjustScanlines << " " << myHeight << endl << std::flush;
myJitterEmulation.setYStart(myYStart); myJitterEmulation.setYStart(myYStart);
} }

View File

@ -52,9 +52,9 @@ class FrameManager: public AbstractFrameManager {
Int32 maxVcenter() const override { return myMaxVcenter; } Int32 maxVcenter() const override { return myMaxVcenter; }
void setAdjustScanlines(float adjustScanlines) override; void setVsizeAdjust(float vsizeAdjust) override;
float adjustScanlines() const override { return myAdjustScanlines; } float vsizeAdjust() const override { return myVsizeAdjust; }
uInt32 startLine() const override { return myYStart; } uInt32 startLine() const override { return myYStart; }
@ -102,7 +102,7 @@ class FrameManager: public AbstractFrameManager {
uInt32 myYStart{0}; uInt32 myYStart{0};
Int32 myVcenter{0}; Int32 myVcenter{0};
Int32 myMaxVcenter{0}; Int32 myMaxVcenter{0};
float myAdjustScanlines{0.f}; float myVsizeAdjust{0.f};
bool myJitterEnabled{false}; bool myJitterEnabled{false};

View File

@ -136,12 +136,12 @@ VideoDialog::VideoDialog(OSystem& osystem, DialogContainer& parent,
ypos += lineHeight + VGAP; ypos += lineHeight + VGAP;
// Aspect ratio (NTSC mode) // Aspect ratio (NTSC mode)
myAdjustScanlines = myVsizeAdjust =
new SliderWidget(myTab, font, xpos, ypos-1, swidth, lineHeight, new SliderWidget(myTab, font, xpos, ypos-1, swidth, lineHeight,
"Scanlines adjust ", lwidth, 0, fontWidth * 4, "", 0, true); "V-Size adjust ", lwidth, 0, fontWidth * 4, "", 0, true);
myAdjustScanlines->setMinValue(-50); myAdjustScanlines->setMaxValue(50); myVsizeAdjust->setMinValue(-50); myVsizeAdjust->setMaxValue(50);
myAdjustScanlines->setTickmarkIntervals(5); myVsizeAdjust->setTickmarkIntervals(5);
wid.push_back(myAdjustScanlines); wid.push_back(myVsizeAdjust);
ypos += lineHeight + VGAP; ypos += lineHeight + VGAP;
// Speed // Speed
@ -342,8 +342,8 @@ void VideoDialog::loadConfig()
myTIAInterpolate->setState(instance().settings().getBool("tia.inter")); myTIAInterpolate->setState(instance().settings().getBool("tia.inter"));
// Aspect ratio setting (NTSC and PAL) // Aspect ratio setting (NTSC and PAL)
myAdjustScanlines->setValue( myVsizeAdjust->setValue(
round(instance().settings().getFloat("tia.adjustscanlines") * 10) round(instance().settings().getFloat("tia.vsizeadjust") * 10)
); );
// Emulation speed // Emulation speed
@ -416,8 +416,8 @@ void VideoDialog::saveConfig()
instance().settings().setValue("tia.inter", myTIAInterpolate->getState()); instance().settings().setValue("tia.inter", myTIAInterpolate->getState());
// Aspect ratio setting (NTSC and PAL) // Aspect ratio setting (NTSC and PAL)
instance().settings().setValue("tia.adjustscanlines", instance().settings().setValue("tia.vsizeadjust",
static_cast<float>(myAdjustScanlines->getValue()) / 10.f static_cast<float>(myVsizeAdjust->getValue()) / 10.f
); );
// Speed // Speed
@ -498,7 +498,7 @@ void VideoDialog::setDefaults()
myTIAZoom->setValue(300); myTIAZoom->setValue(300);
myTIAPalette->setSelected("standard", ""); myTIAPalette->setSelected("standard", "");
myTIAInterpolate->setState(false); myTIAInterpolate->setState(false);
myAdjustScanlines->setValue(0); myVsizeAdjust->setValue(0);
mySpeed->setValue(0); mySpeed->setValue(0);
myFullscreen->setState(false); myFullscreen->setState(false);

View File

@ -58,7 +58,7 @@ class VideoDialog : public Dialog
SliderWidget* myTIAZoom{nullptr}; SliderWidget* myTIAZoom{nullptr};
PopUpWidget* myTIAPalette{nullptr}; PopUpWidget* myTIAPalette{nullptr};
CheckboxWidget* myTIAInterpolate{nullptr}; CheckboxWidget* myTIAInterpolate{nullptr};
SliderWidget* myAdjustScanlines{nullptr}; SliderWidget* myVsizeAdjust{nullptr};
SliderWidget* mySpeed{nullptr}; SliderWidget* mySpeed{nullptr};
CheckboxWidget* myFullscreen{nullptr}; CheckboxWidget* myFullscreen{nullptr};

View File

@ -90,7 +90,7 @@ bool StellaLIBRETRO::create(bool logging)
settings.setValue("tia.zoom", 1); settings.setValue("tia.zoom", 1);
settings.setValue("tia.inter", false); settings.setValue("tia.inter", false);
settings.setValue("tia.", 0.f); settings.setValue("tia.vsizeadjust", 0.f);
//fastscbios //fastscbios
// Fast loading of Supercharger BIOS // Fast loading of Supercharger BIOS