Andrey,
   I made some comments on your review.  Thanks for taking the time to take
a look at the pull request.  I thought your analysis was quite thorough and
it was much appreciated.
   I'm extremely busy at the moment, but I will try to make most of those
modifications and add that to the pull request.  It might be a week or two
before I can get to it.

Thanks again,
scott


On Tue, Jul 16, 2013 at 12:47 AM, dernasherbrezon
<notifications@github.com>wrote:

> In
> maven-fingerprint-plugin/src/main/java/com/st/plugin/fingerprint/FingerprintMojo.java:
>
> > @@ -168,6 +155,9 @@ private StringBuffer processPattern(Pattern p, String data) throws MojoExecution
> >  		Matcher m = p.matcher(data);
> >  		while (m.find()) {
> >  			String curLink = m.group(2);
> > +			for (int i = 0; i < m.groupCount(); ++i) {
>
> should have guard: isDebugEnabled
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dernasherbrezon/maven-fingerprint-plugin/pull/1/files#r5210074>
> .
>