Sounds good to me.
But can you remove all the descendants? The variant header is different for
each one?


On 19 September 2013 16:20, jvelilla <notifications@github.com> wrote:

> In
> library/network/protocol/content_negotation/src/variants/character_encoding_variant_results.e:
>
> > +note
> > +	description: "Summary description for {CHARACTER_ENCODING_VARIANT_RESULTS}."
> > +	author: ""
> > +	date: "$Date$"
> > +	revision: "$Revision$"
> > +
> > +class
> > +	CHARACTER_ENCODING_VARIANT_RESULTS
> > +
> > +inherit
> > +
> > +	VARIANT_RESULTS
> > +	
> > +feature -- Access
> > +
> > +	character_type: detachable READABLE_STRING_8
>
> I agree we can have a single routine set_type, and maybe we can create an
> ACCEPT_HEADER_ENUM and remove all the descendants.
> So the signature could be
>
> set_type (a_accept_header: ACCEPT_HEADER_ENUM)
> -- Set type' witha_accept_header'
>
> What do you think?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/EiffelWebFramework/EWF/pull/71/files#r6463127>
> .
>