vgritsenko 01/06/21 19:26:59
Modified: src/org/apache/cocoon/sitemap Tag: cocoon_20_branch
ContentAggregator.java AbstractSitemap.java
Log:
code cleanup, small optimizations
Revision Changes Path
No revision
No revision
1.5.2.4 +4 -1 xml-cocoon2/src/org/apache/cocoon/sitemap/ContentAggregator.java
Index: ContentAggregator.java
===================================================================
RCS file:
/home/cvs/xml-cocoon2/src/org/apache/cocoon/sitemap/ContentAggregator.java,v
retrieving revision 1.5.2.3
retrieving revision 1.5.2.4
diff -u -r1.5.2.3 -r1.5.2.4
--- ContentAggregator.java 2001/06/12 14:22:31 1.5.2.3
+++ ContentAggregator.java 2001/06/22 02:26:57 1.5.2.4
@@ -42,7 +42,7 @@
/**
* @author <a href="mailto:[EMAIL PROTECTED]";>Giacomo Pati</a>
- * @version CVS $Id: ContentAggregator.java,v 1.5.2.3 2001/06/12 14:22:31 bloritsch
Exp $
+ * @version CVS $Id: ContentAggregator.java,v 1.5.2.4 2001/06/22 02:26:57
vgritsenko Exp $
*/
public class ContentAggregator extends ContentHandlerWrapper implements Generator,
Cacheable, Composable {
/** the current sitemap */
@@ -158,6 +158,9 @@
try {
this.environment.pushURI(part.uri);
ep.process(this.environment);
+ } catch (ProcessingException e) {
+ // Already reported to log, rethrowing.
+ throw e;
} catch (Exception e) {
getLogger().error("ContentAggregator: cannot process event
pipeline for URI " + part.uri, e);
throw new ProcessingException("ContentAggregator: cannot
process event pipeline for URI " + part.uri, e);
1.4.2.10 +6 -8 xml-cocoon2/src/org/apache/cocoon/sitemap/AbstractSitemap.java
Index: AbstractSitemap.java
===================================================================
RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/sitemap/AbstractSitemap.java,v
retrieving revision 1.4.2.9
retrieving revision 1.4.2.10
diff -u -r1.4.2.9 -r1.4.2.10
--- AbstractSitemap.java 2001/06/14 16:43:37 1.4.2.9
+++ AbstractSitemap.java 2001/06/22 02:26:58 1.4.2.10
@@ -43,7 +43,7 @@
* Base class for generated <code>Sitemap</code> classes
*
* @author <a href="mailto:[EMAIL PROTECTED]";>Giacomo Pati</a>
- * @version CVS $Revision: 1.4.2.9 $ $Date: 2001/06/14 16:43:37 $
+ * @version CVS $Revision: 1.4.2.10 $ $Date: 2001/06/22 02:26:58 $
*/
public abstract class AbstractSitemap extends AbstractLoggable implements Sitemap,
Disposable, ThreadSafe {
private Context context;
@@ -274,20 +274,18 @@
throw new PatternException("invalid expression in \"" + expr +
"\"");
}
ii = j + 1;
- if (j == -1) {
- throw new PatternException("invalid expression in URL " + expr);
- }
k = list.size() - 1;
s = expr.substring(i + 1, j);
- l = -1;
- while ((l = s.indexOf("../", l + 1)) != -1) {
+ l = -3;
+ m = -1;
+ while ((l = s.indexOf("../", l + 3)) != -1) {
k--;
+ m = l;
}
- m = s.lastIndexOf('/');
if (m == -1) {
result.append(((Object)((Map)list.get(k)).get(s)).toString());
} else {
- result.append(((Object)((Map)list.get(k)).get(s.substring(m +
1))).toString());
+ result.append(((Object)((Map)list.get(k)).get(s.substring(m +
3))).toString());
}
getLogger().debug("substitute evaluated value for " + ((m == -1) ?
s : s.substring(m + 1)) + " as " +
((Object)((Map)list.get(k)).get((m == -1) ? s : s.substring(m +
1))).toString());
----------------------------------------------------------------------
In case of troubles, e-mail: [EMAIL PROTECTED]
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
