You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by Markus Wiederkehr <ma...@gmail.com> on 2008/11/17 13:23:23 UTC
[mime4j] Remove StringArrayMap?
Does org.apache.james.mime4j.util.StringArrayMap serve any purpose? It
does not seem to be used anywhere and in my opinion should be removed
or marked as deprecated..
Markus
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org
Re: [mime4j] Remove StringArrayMap?
Posted by Markus Wiederkehr <ma...@gmail.com>.
On Mon, Dec 1, 2008 at 2:38 PM, Robert Burrell Donkin
<ro...@gmail.com> wrote:
> On Mon, Dec 1, 2008 at 1:10 PM, Markus Wiederkehr
> <ma...@gmail.com> wrote:
>> On Mon, Nov 17, 2008 at 1:23 PM, Markus Wiederkehr
>> <ma...@gmail.com> wrote:
>>> Does org.apache.james.mime4j.util.StringArrayMap serve any purpose? It
>>> does not seem to be used anywhere and in my opinion should be removed
>>> or marked as deprecated..
>>
>> I have just stumbled upon MIME4J-24. Maybe this class serves a purpose
>> after all? I don't know, you have to decide.
>
> At the very least, it needs to be better documented
I have reverted the deprecation and added a bit more documentation
including a reference to MIME4J-24.
>> If the class should be kept I would at least suggest some refactoring.
>> All the methods that return Enumerations should be refactored to
>> return Iterators and possibly more like that.
>
> submit a patch ;-)
Sorry, not a priority right now. Besides I don't want to break
anything in Commons Fileupload..
Markus
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org
Re: [mime4j] Remove StringArrayMap?
Posted by Robert Burrell Donkin <ro...@gmail.com>.
On Mon, Dec 1, 2008 at 1:10 PM, Markus Wiederkehr
<ma...@gmail.com> wrote:
> On Mon, Nov 17, 2008 at 1:23 PM, Markus Wiederkehr
> <ma...@gmail.com> wrote:
>> Does org.apache.james.mime4j.util.StringArrayMap serve any purpose? It
>> does not seem to be used anywhere and in my opinion should be removed
>> or marked as deprecated..
>
> I have just stumbled upon MIME4J-24. Maybe this class serves a purpose
> after all? I don't know, you have to decide.
At the very least, it needs to be better documented
> If the class should be kept I would at least suggest some refactoring.
> All the methods that return Enumerations should be refactored to
> return Iterators and possibly more like that.
submit a patch ;-)
- robert
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org
Re: [mime4j] Remove StringArrayMap?
Posted by Markus Wiederkehr <ma...@gmail.com>.
On Mon, Nov 17, 2008 at 1:23 PM, Markus Wiederkehr
<ma...@gmail.com> wrote:
> Does org.apache.james.mime4j.util.StringArrayMap serve any purpose? It
> does not seem to be used anywhere and in my opinion should be removed
> or marked as deprecated..
I have just stumbled upon MIME4J-24. Maybe this class serves a purpose
after all? I don't know, you have to decide.
If the class should be kept I would at least suggest some refactoring.
All the methods that return Enumerations should be refactored to
return Iterators and possibly more like that.
Markus
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org
Re: [mime4j] Remove StringArrayMap?
Posted by Oleg Kalnichevski <ol...@apache.org>.
Deprecated. To be removed in 0.7.
Oleg
On Mon, 2008-11-17 at 19:42 +0100, Norman Maurer wrote:
> I think it would worth to deprecate it first...
>
> Cheers,
> Norman
>
> 2008/11/17 Robert Burrell Donkin <ro...@gmail.com>
>
> > On Mon, Nov 17, 2008 at 1:27 PM, Oleg Kalnichevski <ol...@apache.org>
> > wrote:
> > > On Mon, 2008-11-17 at 13:23 +0100, Markus Wiederkehr wrote:
> > >> Does org.apache.james.mime4j.util.StringArrayMap serve any purpose? It
> > >> does not seem to be used anywhere and in my opinion should be removed
> > >> or marked as deprecated..
> > >>
> > >
> > > +1
> >
> > +1
> >
> > if it's removed, best to add a few words to the RELEASE_NOTES
> >
> > - robert
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
> > For additional commands, e-mail: server-dev-help@james.apache.org
> >
> >
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org
Re: [mime4j] Remove StringArrayMap?
Posted by Norman Maurer <no...@apache.org>.
I think it would worth to deprecate it first...
Cheers,
Norman
2008/11/17 Robert Burrell Donkin <ro...@gmail.com>
> On Mon, Nov 17, 2008 at 1:27 PM, Oleg Kalnichevski <ol...@apache.org>
> wrote:
> > On Mon, 2008-11-17 at 13:23 +0100, Markus Wiederkehr wrote:
> >> Does org.apache.james.mime4j.util.StringArrayMap serve any purpose? It
> >> does not seem to be used anywhere and in my opinion should be removed
> >> or marked as deprecated..
> >>
> >
> > +1
>
> +1
>
> if it's removed, best to add a few words to the RELEASE_NOTES
>
> - robert
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
> For additional commands, e-mail: server-dev-help@james.apache.org
>
>
Re: [mime4j] Remove StringArrayMap?
Posted by Robert Burrell Donkin <ro...@gmail.com>.
On Mon, Nov 17, 2008 at 1:27 PM, Oleg Kalnichevski <ol...@apache.org> wrote:
> On Mon, 2008-11-17 at 13:23 +0100, Markus Wiederkehr wrote:
>> Does org.apache.james.mime4j.util.StringArrayMap serve any purpose? It
>> does not seem to be used anywhere and in my opinion should be removed
>> or marked as deprecated..
>>
>
> +1
+1
if it's removed, best to add a few words to the RELEASE_NOTES
- robert
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org
Re: [mime4j] Remove StringArrayMap?
Posted by Oleg Kalnichevski <ol...@apache.org>.
On Mon, 2008-11-17 at 13:23 +0100, Markus Wiederkehr wrote:
> Does org.apache.james.mime4j.util.StringArrayMap serve any purpose? It
> does not seem to be used anywhere and in my opinion should be removed
> or marked as deprecated..
>
+1
Oleg
> Markus
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
> For additional commands, e-mail: server-dev-help@james.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org