Mailing-List: contact dev-help@poi.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "POI Developers List" <dev@poi.apache.org>
From: bugzilla@apache.org
To: dev@poi.apache.org
Subject: [Bug 60795] Add enum for msg types
Date: Fri, 03 Mar 2017 14:35:09 +0000
Message-ID: <bug-60795-47293-Icgix2wGlj@https.bz.apache.org/bugzilla/>
In-Reply-To: <bug-60795-47293@https.bz.apache.org/bugzilla/>
References: <bug-60795-47293@https.bz.apache.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Auto-Submitted: auto-generated
MIME-Version: 1.0
archived-at: Fri, 03 Mar 2017 14:35:14 -0000

https://bz.apache.org/bugzilla/show_bug.cgi?id=3D60795

--- Comment #4 from Tim Allison <tallison@mitre.org> ---
Javen,
  I'm sorry for my delay.

> Will we be able to remove/replace String getMessageClass() with getMessag=
eClassEnum()?

Yes, and then back again much later, right?  The user can still get the
underlying StringChunk if they want the literal chunk.

> The enum version calls the string version (for now). This would mean we'd=
 have to move some code around or rename public String getMessageClass () t=
o private String getMessageClassString().

I fixed that.  Thank you.

> mainChunks.getMessageClass() looks like this method is part of an interfa=
ce. Have we deprecated that method in the interface and all implementing cl=
asses?

I don't think we should.  That is a low-level call that returns the
StringChunk.  Advanced users should have access to that.

--=20
You are receiving this mail because:
You are the assignee for the bug.=

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org