You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2020/01/31 16:07:53 UTC
[tomcat] branch master updated: Deprecate MappingData.contextPath
as it is unused
This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/master by this push:
new a4d54ca Deprecate MappingData.contextPath as it is unused
a4d54ca is described below
commit a4d54cab7b30d380d31a23fc3844e3da90cace5c
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Fri Jan 31 16:07:15 2020 +0000
Deprecate MappingData.contextPath as it is unused
---
java/org/apache/catalina/mapper/Mapper.java | 2 ++
java/org/apache/catalina/mapper/MappingData.java | 4 ++++
test/org/apache/catalina/mapper/TestMapper.java | 2 ++
3 files changed, 8 insertions(+)
diff --git a/java/org/apache/catalina/mapper/Mapper.java b/java/org/apache/catalina/mapper/Mapper.java
index a584567..0553f4a 100644
--- a/java/org/apache/catalina/mapper/Mapper.java
+++ b/java/org/apache/catalina/mapper/Mapper.java
@@ -732,6 +732,7 @@ public final class Mapper {
* Map the specified URI.
* @throws IOException
*/
+ @SuppressWarnings("deprecation") // contextPath
private final void internalMap(CharChunk host, CharChunk uri,
String version, MappingData mappingData) throws IOException {
@@ -1064,6 +1065,7 @@ public final class Mapper {
/**
* Exact mapping.
*/
+ @SuppressWarnings("deprecation") // contextPath
private final void internalMapExactWrapper
(MappedWrapper[] wrappers, CharChunk path, MappingData mappingData) {
MappedWrapper wrapper = exactFind(wrappers, path);
diff --git a/java/org/apache/catalina/mapper/MappingData.java b/java/org/apache/catalina/mapper/MappingData.java
index 4480854..2d9ec3a 100644
--- a/java/org/apache/catalina/mapper/MappingData.java
+++ b/java/org/apache/catalina/mapper/MappingData.java
@@ -38,6 +38,10 @@ public class MappingData {
public Wrapper wrapper = null;
public boolean jspWildCard = false;
+ /**
+ * @deprecated Unused. This will be removed in Tomcat 10.
+ */
+ @Deprecated
public final MessageBytes contextPath = MessageBytes.newInstance();
public final MessageBytes requestPath = MessageBytes.newInstance();
public final MessageBytes wrapperPath = MessageBytes.newInstance();
diff --git a/test/org/apache/catalina/mapper/TestMapper.java b/test/org/apache/catalina/mapper/TestMapper.java
index f155f10..23a08d1 100644
--- a/test/org/apache/catalina/mapper/TestMapper.java
+++ b/test/org/apache/catalina/mapper/TestMapper.java
@@ -241,6 +241,7 @@ public class TestMapper extends LoggingBaseTest {
}
@Test
+ @SuppressWarnings("deprecation") // contextPath
public void testMap() throws Exception {
MappingData mappingData = new MappingData();
MessageBytes host = MessageBytes.newInstance();
@@ -492,6 +493,7 @@ public class TestMapper extends LoggingBaseTest {
}
@Test
+ @SuppressWarnings("deprecation") // contextPath
public void testContextListConcurrencyBug56653() throws Exception {
final Host host = createHost("localhost");
final Context contextRoot = createContext("ROOT");
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org