Skip to end of metadata
Go to start of metadata

You are viewing an old version of this page. View the current version.

Compare with Current View Page History

« Previous Version 7 Next »

[13:00:44 CDT(-0500)] <EricDalquist> The String.split method annoys me

[13:00:56 CDT(-0500)] <EricDalquist> every time you call it you're compiling a new Pattern

[13:01:04 CDT(-0500)] <EricDalquist> so it is useful

[13:01:07 CDT(-0500)] <EricDalquist> it saves a little code

[13:01:10 CDT(-0500)] <EricDalquist> but used in hot spots

[13:01:17 CDT(-0500)] <EricDalquist> it really chews CPU

[13:07:52 CDT(-0500)] <athena> yeah

[13:07:57 CDT(-0500)] <athena> that's a good point

[13:08:08 CDT(-0500)] <athena> can you feed it a pre-compiled pattern?

[13:08:14 CDT(-0500)] <EricDalquist> no but you can do:

[13:08:27 CDT(-0500)] <EricDalquist> p = Pattern.compile(...);

[13:08:33 CDT(-0500)] <EricDalquist> p.split(String)

[13:08:40 CDT(-0500)] <EricDalquist> and store p as a static final

[13:08:47 CDT(-0500)] <athena> that makes sense

[13:09:05 CDT(-0500)] <EricDalquist> maybe someday uPortal's code will be clean enough that we can add checkstyle (smile)

[13:11:25 CDT(-0500)] <athena> (smile)

[13:42:31 CDT(-0500)] <JoeMoore> Im trying to configure portlet parameters. It wont let me edit them at the beginning, but when I try to edit them with the rich editor I get http://pastebin.com/Q5Y9j5Cw

[14:50:19 CDT(-0500)] <JoeMoore> I guess my pastebin timed out (one hour). Let me know if someone wants to take a look and I'll paste again with a longer timeout on the problem of configuring portlet parameters.

  • No labels