[threeten-dev] Non-webrev: Removing builder

[Stephen Colebourne]

I've pushed the main change here, which removes builder from the
public API of TemporalField. There is still more work to do, but this
code was worth pushing anyway. (Normally I'd use a personal branch,
but that option isn't available in OpenJDK hg)

This code also fixes a non-deterministic infinite loop in WeekFields.

The remaining work is to remove builder from the special cases in
LocalDate and LocalTime, then make the class non-public or remove it.

Ideally, I'd like to replace parseUnresolved by something more hidden
but I may not have time for that.

Stephen



On 28 January 2013 01:14, Stephen Colebourne <scolebourne at joda.org> wrote:
> This is the start of a patch to remove DateTimeBuilder. It isn't
> complete but can be applied and does work without changing behaviour.
> I believe the old resolve method on TemporalField will be able to be
> removed.
>
> This is for info and to indicate a direction of travel rather than for
> full review.
>
> Stephen