TVPT / VoxelSniper

The premiere long-distance brush editor for Minecraft
Other
128 stars 115 forks source link

Re-add Spline Brush; adjust language to be more consistent and correct. #175

Closed Aerodog closed 11 years ago

Aerodog commented 11 years ago

I meant to divide this into two commits, but I accidentally committed all of the staged changes at once. Is this okay?

ayyy commented 11 years ago

Does the spline brush work? I believe someone said that it didn't work and was removed because of that.

Aerodog commented 11 years ago

It works perfectly fine and I use it to build curved elements, such as bridges, but when you undo a spline snipe, some blocks remain. I don't think that's enough reason for it to be removed. Even if you don't use the spline generated, it's an excellent skeleton/outline maker. The undo bug affects other brushes that currently exist anyway.

People may have thought it was bugged because the curve generated doesn't reach the control points, but that is intended. The spline uses the control points as guides/suggestions and extrapolates a curve. It does not and cannot include both control points in the spline. This might not make sense if you haven't used the brush, but to make a long story short, I'm 99% certain that its removal was the result of a misunderstanding.

MikeMatrix commented 11 years ago

Merged. Thanks.