Permalink
Browse files

DINGUX: Remove dead code

When compiled with C++11, it was revealed that initSize has the
wrong method signature so has not been overriding initSize
and so has not been doing anything. Apparently this is OK, since
nobody has complained about it, so just get rid of it.

The setGraphicsMode(const char *) signature was also wrong, and
has no implementation code anyway.
  • Loading branch information...
csnover authored and ccawley2011 committed Dec 5, 2017
1 parent df236f1 commit a7ddf1dc0d26c6823af74a324ea281b6fa79127b
@@ -117,24 +117,6 @@ void DINGUXSdlGraphicsManager::setGraphicsModeIntern() {
blitCursor();
}

void DINGUXSdlGraphicsManager::initSize(uint w, uint h) {
assert(_transactionMode == kTransactionActive);

// Avoid redundant res changes
if ((int)w == _videoMode.screenWidth && (int)h == _videoMode.screenHeight)
return;

_videoMode.screenWidth = w;
_videoMode.screenHeight = h;
if (w > 320 || h > 240) {
setGraphicsMode(GFX_HALF);
setGraphicsModeIntern();
_window->toggleMouseGrab();
}

_transactionDetails.sizeChanged = true;
}

void DINGUXSdlGraphicsManager::drawMouse() {
if (!_cursorVisible || !_mouseSurface) {
_mouseBackup.x = _mouseBackup.y = _mouseBackup.w = _mouseBackup.h = 0;
@@ -41,9 +41,7 @@ class DINGUXSdlGraphicsManager : public SurfaceSdlGraphicsManager {
bool getFeatureState(OSystem::Feature f) const override;
int getDefaultGraphicsMode() const override;

void initSize(uint w, uint h) override;
const OSystem::GraphicsMode *getSupportedGraphicsModes() const override;
bool setGraphicsMode(const char *name) override;
bool setGraphicsMode(int mode) override;
void setGraphicsModeIntern() override;
void internUpdateScreen() override;

0 comments on commit a7ddf1d

Please sign in to comment.