I must say I don't remember exactly. I did many test and research and
compare with what others trying to solve the same in the internet without
using a concrete library that at that time I think we want to avoid. So you
can try to compile with and without and use a compare tool to see what you
get and maybe that show the intention of the regex . Maybe it make some
little improvement, but if it remove something, better remove it.

2018-06-06 14:48 GMT+02:00 Harbs <[email protected]>:

> I understand, but I’m trying to understand what the logic of that line of
> code was. What was it trying to do?
>
> I could simply remove the line, but if it’s serving a function, we should
> probably fix the RegEx instead.
>
> Thanks,
> Harbs
>
> > On Jun 6, 2018, at 3:36 PM, Carlos Rovira <[email protected]>
> wrote:
> >
> > Hi Harbs,
> >
> > that was a simple attempt to minify css code, if that line is not working
> > you can comment it, and we'll be improving the minifying method.
> >
> > thanks
> >
> >
> >
> > 2018-06-06 14:27 GMT+02:00 Harbs <[email protected]>:
> >
> >> It’s this code in minifyCSSString:
> >>        cssString = cssString.replaceAll("[a-zA-Z]+#", "#");
> >>
> >> Why is that code there?
> >>
> >> Harbs
>
>


-- 
Carlos Rovira
http://about.me/carlosrovira

Reply via email to