From 2af508ef838daf87bad66a9db129e499194ed486 Mon Sep 17 00:00:00 2001 From: ebanks Date: Mon, 15 Nov 2010 18:24:15 +0000 Subject: [PATCH] Better docs, as requested by Matt git-svn-id: file:///humgen/gsa-scr1/gsa-engineering/svn_contents/trunk@4681 348d0f76-0448-11de-a6fe-93d51630548a --- .../broadinstitute/sting/commandline/CommandLineUtils.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/java/src/org/broadinstitute/sting/commandline/CommandLineUtils.java b/java/src/org/broadinstitute/sting/commandline/CommandLineUtils.java index feaec282c..57e83a38c 100644 --- a/java/src/org/broadinstitute/sting/commandline/CommandLineUtils.java +++ b/java/src/org/broadinstitute/sting/commandline/CommandLineUtils.java @@ -133,7 +133,11 @@ public class CommandLineUtils { } } - // TODO -- is there a better way to do this? + // The problem here is that some of the fields being output are Objects - and those + // Objects don't overload toString() so that the output is just the memory pointer + // to the Object. Because those values are non-deterministic, they don't merge well + // into BAM/VCF headers (plus, it's just damn ugly). Perhaps there's a better way to + // do this, but at least this one works for the moment. private static final String pointerRegexp = ".+@[0-9a-fA-F]+$"; private static boolean isObjectPointer(String s) { return s != null && s.matches(pointerRegexp);