TASK: Correct support for Windows path

Author: dfeyer

flowpathmapper/flowpathmapper.go

@@ -29,11 +29,8 @@ const (
 
 var (
 	regexpPhpFile         = regexp.MustCompile(`(?://)?(/[^ ]*\.php)`)
-	if runtime.GOOS == "windows" {
-		regexpFilename        = regexp.MustCompile(`filename=["]?file:///(\S+)/Data/Temporary/.+?/Cache/Code/Flow_Object_Classes/([^"]*)\.php`)
-	} else
-		regexpFilename        = regexp.MustCompile(`filename=["]?file://(\S+)/Data/Temporary/.+?/Cache/Code/Flow_Object_Classes/([^"]*)\.php`)
-	}
+	regexpFilename__Win   = regexp.MustCompile(`filename=["]?file:///(\S+)/Data/Temporary/.+?/Cache/Code/Flow_Object_Classes/([^"]*)\.php`)
+	regexpFilename__Unix  = regexp.MustCompile(`filename=["]?file://(\S+)/Data/Temporary/.+?/Cache/Code/Flow_Object_Classes/([^"]*)\.php`)
 	regexpPathAndFilename = regexp.MustCompile(`(?m)^# PathAndFilename: (.*)$`)
 	regexpPackageClass    = regexp.MustCompile(`(.*?)/Packages/[^/]*/(.*?)/Classes/(.*).php`)
 	regexpDot             = regexp.MustCompile(`[\./]`)
@@ -44,6 +41,13 @@ func init() {
 	pathmapperfactory.Register(framework, p)
 }
 
+func regexpFilename() *regexp.Regexp {
+	if runtime.GOOS == "windows" {
+		return regexpFilename__Win
+	}
+	return regexpFilename__Unix
+}
+
 // PathMapper handle the mapping between real code and proxy
 type PathMapper struct {
 	config      *config.Config
@@ -112,7 +116,7 @@ func (p *PathMapper) getCachePath(base, filename string) string {
 
 func (p *PathMapper) doXMLPathMapping(b []byte) []byte {
 	var processedMapping = map[string]string{}
-	for _, match := range regexpFilename.FindAllStringSubmatch(string(b), -1) {
+	for _, match := range regexpFilename().FindAllStringSubmatch(string(b), -1) {
 		path := p.getCachePath(match[1], match[2])
 		if _, ok := processedMapping[path]; ok == false {
 			if originalPath, exist := p.pathMapping.Get(path); exist {