Comments (5)
Seems like you're trying to build the game with the Debug configuratipn. Switch to Release build. The settings for the Debug build are a bit over restrictive.
from openrct2.
Although saying that there is a good point that widget will potentially be not initialized see line 520 (input_mouseover). I think there should probably be an else return but i would have to check the original code. Same for the other function at line 621 (input_leftmousedown). These will need to be fixed at some point so don't close the bug.
from openrct2.
Does this still apply to the current code?
from openrct2.
The code has moved to https://github.com/OpenRCT2/OpenRCT2/blob/develop/src/input.c#L798 and I don't think this warning is valid. I ceratinly haven't seen it with clang/gcc compiles.
in line 798 widget
is declared without explicit assignment, in line 800 if (w != NULL)
then in line 803 widget
has value assigned. In line 808, which was previously 526 mentioned in original report, there is a check if (w != NULL && widgetIndex != -1 && widget->type == WWT_SCROLL)
which dereferences widget
if and only if w != NULL
, which guaranteed value assignment in line 803 above.
An assert could be added just after assignment in line 803 to see if widget is not NULL, but otherwise it looks fine.
from openrct2.
Considering the changes since 2014 and janisozaur's comment I'll assume this no longer applies to the current state of the code.
from openrct2.
Related Issues (20)
- Mango Muncher can't be placed with scenery in Electric Fields HOT 3
- Entity Viewer broke since OpenRCT2 v0.4.1
- [745952] openrct2.exe: OpenRCT2::Ui::Windows::TileInspector::CopyElement
- [f30934] openrct2.exe: protected: CAddressEditBox::_ConnectToBrwsrWnd
- "Trap Mouse Cursor in Window" has buggy behavior with multiple monitors. HOT 4
- Making multiple "stations" for rides that should only have 1
- [441408] openrct2.exe: WindowClose
- [b33871] openrct2.com: ForceCrash
- Lightning turns night into day
- [72751b] openrct2.exe: graphics-hook64.dll
- [6142de] openrct2.exe: duk__get_own_propdesc_raw
- [9985b2] openrct2.exe: IntentData::~IntentData
- [33dbe2] openrct2.exe: IntentData::~IntentData
- Opening "Object Selection" closes all other windows HOT 1
- [509d4c] openrct2.exe: WidgetInvalidate
- macOS builds failing
- [9f18ec] openrct2.exe: duk__hobject_alloc_entry_checked
- [08571d] openrct2.exe: duk_unicode_encode_cesu8
- Modifying path corners via tile inspector does weird things
- Uncap FPS causes a red error to pop up. " Unable to change ride operating mode" HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from openrct2.