CharSequence is a trait which String implements. It probably makes sense to
have MdTag accept CharSequence everywhere instead of String since it's more
general. This will remove any ambiquity.



-Matt


On Mon, Nov 25, 2013 at 9:14 AM, Frank Austin Nothaft <
notifications@github.com> wrote:

> In
> adam-commands/src/main/scala/edu/berkeley/cs/amplab/adam/rich/RichADAMRecord.scala:
>
> > @@ -105,7 +105,7 @@ class RichADAMRecord(record: ADAMRecord) {
> >    }
> >
> >    lazy val mdEvent: Option[MdTag] = if (record.getMismatchingPositions != null) {
> > -    Some(MdTag(record.getMismatchingPositions, record.getStart))
> > +    Some(MdTag(record.getMismatchingPositions.toString, record.getStart))
>
> Correct. However, I don't see this stack overflow in real world use, I
> only see it in the unit test framework.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bigdatagenomics/adam/pull/6/files#r7897163>
> .
>