Duncan

Results 370 comments of Duncan

Well once you have the S5File its already broken into its constituent parts.

https://github.com/OpenRCT2/OpenRCT2/blob/develop/src/openrct2/cmdline/CommandLine.hpp this is what is used for OpenRCT2. Suggest it is copied.

The idea is that the common code would be in [libsawyer](https://github.com/IntelOrca/libsawyer).

Don't think we have done enough of the update code to do those features yet.

> Technically `visitAll` can be achieved by just using `visitFirst`. Is there need for both? It would also be nice to have a `find` and `findIndex` I think. Perhaps it...

> @duncanspumpkin how about just `find` and `visit`, both returning a condition to say whether to terminate the loop. We could at least start off with that, and see how...

> Maybe we could have some sort of method chain... > > ```c++ > auto el = tile.find() > .withZ(25) > .noGhosts() > .where([](auto& b) { return customPredicate; }; >...

> > > Maybe we could have some sort of method chain... > > This is called a [fluent interface](https://en.wikipedia.org/wiki/Fluent_interface) or fluent api. You just have a set of methods...

This sounds more like a scenario option (which would require a new save format) rather than just modifying the existing no breakdown cheat/option.

I think high scores is the only bit of code still using vanilla for saving unfortunately.