gonzalad/core — comment on commit ca9a5b0fe7042efe26db38489a70018c780f8512
File: impl/src/main/java/org/richfaces/resource/ResourceHandlerImpl.java line 65
By lfryc:

Thanks,

1. static lookup is nice way of dealing with context injection to custom classes, but I would avoid it in JSF resources if it isn't specified it is available on that place. It just avoids regressions.

2. accessing FacesContext.getApplication().getProjectStage() should not get any extra overhead here.

3. accesing FacesContext.getCurrentInstance() may take some extra cycles (but since you did it in constructor, it doesn't apply here)