#8997 closed patch (wontfix)
GSOC09: Objectifying CruisE - Trivial Patch #1
Reported by: | SF/remoun | Owned by: | dreammaster |
---|---|---|---|
Priority: | normal | Component: | Engine: Cruise |
Version: | Keywords: | ||
Cc: | Game: |
Description
A trivial, rather hacky refactoring of the Cell struct in the CruisE: * moved the functions in cell.h into cellStruct, replacing the (cellStruct *) parameter with 'this'. * updated calling functions accordingly
Ticket imported from: #2742211. Ticket imported from: patches/1102.
Attachments (1)
Change History (11)
by , 16 years ago
Attachment: | Cruise_Cell.patch added |
---|
comment:1 by , 16 years ago
Made another couple of changes: * renamed cellStruct to Cell * renamed (the global variable) cellHead to g_cellHead * moved freeObjectList from function.h/cpp to Cell * added an addCellAfter function, for linked list-convenience * fixed capitalization of 'currentcellHead' in saveload.cpp
comment:2 by , 16 years ago
Summary: | Objectifying CruisE - Trivial Patch #1 (GSoC Patch) → GSOC09: Objectifying CruisE - Trivial Patch #1 |
---|
comment:5 by , 16 years ago
Owner: | set to |
---|
comment:6 by , 16 years ago
"Patch reviewed" -- OK, but should this patch no be accepted, rejected or revised? :)
Maybe dreammaster (who recently worked on Cruise a bit) can judge it better than us?
comment:7 by , 16 years ago
Owner: | changed from | to
---|
comment:8 by , 16 years ago
It may be a trivial refactoring, but it also causes the engine to hang when integrated. I recall from prior discussions in the group that remoun didn't have a copy of Cruise handy to test the changes on, so it's likely there some fundamental flaw in the changes that were made.
Given that, I'm rejecting the patch for integration. I'd recommend that anyone wanting to objectify the engine do the Cell class again from scratch, and perhaps use standard ScummVM list classes for holding the cells, rather than trying to maintain a doubly-linked list (as this does).
comment:9 by , 16 years ago
Resolution: | → wontfix |
---|---|
Status: | new → closed |
comment:10 by , 6 years ago
Component: | → Engine: Cruise |
---|
Changes to Cruise::cellStruct (now Cruise::Cell), as of SVN trunk rev. 39900