Do not swallow cause of exception about play1 HOT 4 CLOSED

Done in branch 1.4.x for many cases (but unfortunately not for all).

from play1.

While it is a good philosophy to not swallow exceptions, the code now no longer executes the original author's intent.

The code is meant to identify whether the current start tag is a GroovyInlineTag, i.e., an #{if...} #{else...} or #{list...}

If it is not, then the code should swallow the exception and continue to the next section to note other characteristics of the tag.

You should probably put back the original exception handler of:

// do nothing here

And add more code comments so that future maintainers have a clearer understanding of what's happening in this method.

from play1.

@selangley I agree, I enabled too much logging. I am going to remove some of them.

from play1.

@asolntsev It is the problem I talked you about, I will try to keep your change and avoiding the exception: will be handle in #1009

from play1.