At the time the move coloring was implemented we thought an enum of modes
is the best to configure this feature. However as we want to tack on new
features, the enum would grow exponentially.
Refactor the code such that features are enabled via bits. Currently we can
* activate the move detection
On Mon, 2 Apr 2018 15:48:52 -0700
Stefan Beller wrote:
> At the time the move coloring was implemented we thought an enum of modes
> is the best to configure this feature. However as we want to tack on new
> features, the enum would grow exponentially.
>
> Refactor the code such that features
On Mon, Apr 2, 2018 at 4:51 PM, Jonathan Tan wrote:
> On Mon, 2 Apr 2018 15:48:52 -0700
> Stefan Beller wrote:
>
>> At the time the move coloring was implemented we thought an enum of modes
>> is the best to configure this feature. However as we want to tack on new
>> features, the enum would g
On Mon, Apr 02 2018, Stefan Beller wrote:
> At the time the move coloring was implemented we thought an enum of modes
> is the best to configure this feature. However as we want to tack on new
> features, the enum would grow exponentially.
>
> Refactor the code such that features are enabled via
On Tue, Apr 3, 2018 at 12:39 PM, Ævar Arnfjörð Bjarmason
wrote:
>
> On Mon, Apr 02 2018, Stefan Beller wrote:
>
>> At the time the move coloring was implemented we thought an enum of modes
>> is the best to configure this feature. However as we want to tack on new
>> features, the enum would grow
On Tue, Apr 03 2018, Stefan Beller wrote:
> On Tue, Apr 3, 2018 at 12:39 PM, Ævar Arnfjörð Bjarmason
> wrote:
>>
>> On Mon, Apr 02 2018, Stefan Beller wrote:
>>
>>> At the time the move coloring was implemented we thought an enum of modes
>>> is the best to configure this feature. However as we
On Mon, Apr 2, 2018 at 4:51 PM, Jonathan Tan wrote:
> On Mon, 2 Apr 2018 15:48:52 -0700
> Stefan Beller wrote:
>
>> At the time the move coloring was implemented we thought an enum of modes
>> is the best to configure this feature. However as we want to tack on new
>> features, the enum would g
On Fri, 6 Apr 2018 14:28:40 -0700
Stefan Beller wrote:
> Now that I redid it another way[1], I have the impression that this was the
> right approach, because it allows for a short
> if (o->color_moved)
> condition. If we treat white spaces separately, then we'd have to
> have implications such
8 matches
Mail list logo