Will do.

On 31 Jul 2013, at 01:26, Christopher Orr <notifications@github.com> wrote:

> In src/main/java/hudson/plugins/junitattachments/GetTestDataMethodObject.java:
> 
> > +				// compute the file name
> > +				line = line.substring(PREFIX.length(),line.length()-SUFFIX.length());
> > +				int idx = line.indexOf('|');
> > +				if (idx>=0) line = line.substring(0,idx);
> > +
> > +				String fileName = line;
> > +				if (fileName!=null) {
> > +					FilePath src = build.getWorkspace().child(fileName);   // even though we use child(), this should be absolute
> > +					if (src.exists()) {
> > +						captureAttachment(className, testName, src);
> > +					} else {
> > +						listener.getLogger().println("Attachment "+fileName+" was referenced from the test '"+className+"' but it doesn't exist. Skipping.");
> > +					}
> > +				}
> > +			}
> > +		}
> Could you possibly keep the original file formatting, so that it's clear to see here what has changed?
> 
> —
> Reply to this email directly or view it on GitHub.
>