search

TheDreamSanctuary / Multitool

GNU General Public License v2.0
0 stars 2 forks source link

Handler alternative #16

Closed DukeVindzor closed 9 years ago

DukeVindzor commented 9 years ago

I have been testing the handler with both the sledgehammer and jackhammer. Both of the tools use all the information provided differently in this tool handler compared to the current handler, meaning there should not be any problems for future tools.

Additionally, the sledgehammer does all the tasks both paint brush and data wrench will do. Therefore I will go as far as to say this handler is safe to use. Naturally, more thorough testing will be needed towards the end of the project. If it turns out my tool handler does not provide the needed support, I will take responsibility and do any changes necessary to revert back to the current handler (switching between the handlers should be quite easy).

I want you guys' opinion before merging :)

AceDexter commented 9 years ago

I mean, as long as it works I don't see a problem with it ^_^ So with the Jackhammer and Sledgehammer working I take it the Paintbrush and Data Wrench aren't too far off in development?

DukeVindzor commented 9 years ago

You are assigned paint brush and data wrench has not been assigned anyone yet, so I haven't done anything on those, but with the new BlockDataHelper class making the paintbrush should be quite a bit easier :)

As you agree on merging, I will go ahead and merge.