[/musicextras/musicextras: changeset 14
Zachary P. Landau <kapheine@divineinvasion.net>**20040924043111
 Put the most common regexp into a variable.
] {
hunk ./lib/musicextras/musicsites/allmusic.rb 44
+    COMMON_REGEX = '<a href="\/cg\/amg.dll\?p=amg&sql=([^"]*)">([^<]*)<\/a>'
hunk ./lib/musicextras/musicsites/allmusic.rb 160
-      #ma = review_page.match(/class=\"title\">Review<\/td><td align="right" class="author">by\s*?([^<]*)<\/td>.*?<p>(.*?)<!--End Center Content/mi)
hunk ./lib/musicextras/musicsites/allmusic.rb 202
-	body.scan(/<a href="\/cg\/amg.dll\?p=amg&sql=(11:[^"]*)">([^<]*)<\/a>/mi) do |url, name|
-          debug_var { :url }
+	body.scan(/#{COMMON_REGEX}/mi) do |url, name|
+	  debug_var { :url }
hunk ./lib/musicextras/musicsites/allmusic.rb 256
-	  page.scan(/<a href="\/cg\/amg.dll\?p=amg&sql=(10:[^"]*)">([^<]*)<\/a>/mi) do |url, name|
+	  page.scan(/#{COMMON_REGEX}/mi) do |url, name|
hunk ./lib/musicextras/musicsites/allmusic.rb 305
-      tracks = ""
+      tracks = "\n"
hunk ./lib/musicextras/musicsites/allmusic.rb 307
-      page.scan(/<TD class="cell">(\d*?)<\/TD>.*?<a href="\/cg\/amg.dll\?p=amg&sql=([^"]*)">([^<]*)<\/a>/mi) do |num, name|
+      page.scan(/<TD class="cell">(\d*?)<\/TD>.*?<a href="\/cg\/amg.dll\?p=amg&sql=[^"]*">([^<]*)<\/a>/mi) do |num, name|
}