Is There A Better (Quicker Processing) Way To Write This?

[johnsmith153]

This:

 

/[^\(]+\.(gif|jpg|jpeg|png|svg|bmp)[^;\s]*\)/gi

 

..runs quite slow.

 

Any ideas to speed it up?

[.josh]

it would help if you gave example content and exactly what you are trying to get out of it. 

[johnsmith153]

Hi,

 

Thanks for replying.

 

My Regex is very poor and I put this together the best I can using various tutorials etc.

 

If anything looks stupid then it probably is.

 

It basically searches through a CSS file and returns an array of image URLs. That's all it needs to do. It doesn't need to to replace anything. It works, but takes ages to complete the processing.

[.josh]

it would help if you gave example content and exactly what you are trying to get out of it. 

 

 

[johnsmith153]

Example content: a CSS file, such as http://www.phpfreaks.com/media/css/screen.min.css

 

It basically searches through a CSS file and returns an array of image URLs.

[.josh]

Okay well, you still aren't being terribly helpful. I'm not much of a css person so I don't know all the syntax rules/possibilities for image URLs in a css file. Never assume that the context you are working with is common knowledge...

 

Having said that..having a quick look at the "example" you provided, I would go with this:

 

/url\s?\(([^)]*)/gi