View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0000030 | Multi Theft Auto | Other | public | 2012-07-03 11:09 | 2022-01-06 04:23 |
| Reporter | Callum | Assigned To | SugarD-x | ||
| Priority | urgent | Severity | major | Reproducibility | N/A |
| Status | closed | Resolution | no change required | ||
| Target Version | 0.6.1 | Fixed in Version | 0.6 | ||
| Summary | 0000030: Replace structs with classes | ||||
| Description | Replace all C structs with classes, in order to modernize coding, and hopefully fix all these crashes. It's a very big job though. Should start with a CPlayer class. | ||||
| Tags | No tags attached. | ||||
|
|
Doing this will require rewriting a lot of the code, so it can be done for 0.6. |
|
|
This is such a big change that you might as well stop using SCM completely. Not something I would do. It would only break code. Changes have be made slowly and carefully. It would also be a waste of time to just 'replace structs with classes' - if I was to do that, I would go way beyond, but it's simply not doable. |
|
|
What is the consensus on this for 0.6? If this requires a rewrite, we may want to save it for 0.7 since 0.6 needs to open-source the code. |
|
|
No please, NO NOO. This gonna cause horrible damage, Rule 0000001 in legacy code don't change that much. |
|
|
Due to the complexity involved and the lack of relativity to this particular version, this will be closed for now, however can be revisited for a later version. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2012-07-03 11:09 | Callum | New Issue | |
| 2012-11-25 12:48 | Callum | Note Added: 0000027 | |
| 2012-11-25 12:48 | Callum | Target Version | 0.6 => 0.6.1 |
| 2015-07-21 04:49 | ixjf | Note Added: 0000073 | |
| 2015-07-21 04:50 | ixjf | Note Edited: 0000073 | |
| 2018-01-06 17:35 | SugarD-x | Assigned To | => SugarD-x |
| 2018-01-06 17:35 | SugarD-x | Status | new => feedback |
| 2018-01-06 17:35 | SugarD-x | Assigned To | SugarD-x => |
| 2018-01-06 17:36 | SugarD-x | Note Added: 0000106 | |
| 2022-01-06 04:22 | Zurix | Note Added: 0000198 | |
| 2022-01-06 04:23 | SugarD-x | Assigned To | => SugarD-x |
| 2022-01-06 04:23 | SugarD-x | Status | feedback => closed |
| 2022-01-06 04:23 | SugarD-x | Resolution | open => no change required |
| 2022-01-06 04:23 | SugarD-x | Fixed in Version | => 0.6 |
| 2022-01-06 04:23 | SugarD-x | Note Added: 0000199 |
