search

OutpostUniverse / op2ext

Outpost 2 extension module loader
1 stars 0 forks source link

Use `vector` for `AddVolToList` #253

Closed DanRStevens closed 5 years ago

DanRStevens commented 5 years ago

This is a bit of background work for issue #239. This collects all uses of VolList into the OnInit() method in DllMain.cpp.

The longer term plan is to rework OnInit() to build the full VOL list first, and then pass the full list to a new VolList constructor. Currently the code is still constructing an empty VolList and then adding VOL files one at a time.

I'm stopping here to come up with a plan for the next part. The VOL list building could be done in one of many ways, and all of them are a bit awkward with current C++ features.

DanRStevens commented 5 years ago

I know what you mean. I was tempted to not push it until I'd come up with a plan for the rest of it.