Move Regexp so that we only compile it once.

rubenv · rubenv · commit 7bb8b6b53518 · 2014-12-14T09:36:01.000+01:00
diff --git a/migrate.go b/migrate.go
@@ -26,6 +26,7 @@ const (
 )
 
 var tableName = "gorp_migrations"
+var numberPrefixRegex = regexp.MustCompile(`^(\d+).*$`)
 
 // Set the name of the table used to store migration info.
 //
@@ -60,8 +61,7 @@ func (m Migration) isNumeric() bool {
 }
 
 func (m Migration) NumberPrefixMatches() []string {
-	r := regexp.MustCompile(`^(\d+).*$`)
-	return r.FindStringSubmatch(m.Id)
+	return numberPrefixRegex.FindStringSubmatch(m.Id)
 }
 
 func (m Migration) VersionInt() int64 {

Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,7 @@ const (`
`26`	`26`	`)`
`27`	`27`
`28`	`28`	`var tableName = "gorp_migrations"`
	`29`	+var numberPrefixRegex = regexp.MustCompile(`^(\d+).*$`)
`29`	`30`
`30`	`31`	`// Set the name of the table used to store migration info.`
`31`	`32`	`//`
`@@ -60,8 +61,7 @@ func (m Migration) isNumeric() bool {`
`60`	`61`	`}`
`61`	`62`
`62`	`63`	`func (m Migration) NumberPrefixMatches() []string {`
`63`		- r := regexp.MustCompile(`^(\d+).*$`)
`64`		`- return r.FindStringSubmatch(m.Id)`
	`64`	`+ return numberPrefixRegex.FindStringSubmatch(m.Id)`
`65`	`65`	`}`
`66`	`66`
`67`	`67`	`func (m Migration) VersionInt() int64 {`