Skip to content

Commit 05e5643

Browse files
committed
Recover STARHACK, avoid deprecation warnings
1 parent 79c356e commit 05e5643

File tree

3 files changed

+18
-19
lines changed

3 files changed

+18
-19
lines changed

ci/io/github/css4j/ci/SampleSitesIT.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
import java.io.InputStreamReader;
2323
import java.io.Reader;
2424
import java.io.StringReader;
25+
import java.net.URISyntaxException;
2526
import java.net.URL;
2627
import java.net.URLConnection;
2728
import java.nio.charset.StandardCharsets;
@@ -214,6 +215,11 @@ public class SampleSitesIT {
214215
}
215216

216217
// NSAC flags
218+
s = config.getProperty("parser.starhack");
219+
if ("true".equalsIgnoreCase(s)) {
220+
parserFlags.add(Parser.Flag.STARHACK);
221+
log.info("IE star hack allowed.");
222+
}
217223
s = config.getProperty("parser.ievalues");
218224
if ("true".equalsIgnoreCase(s)) {
219225
parserFlags.add(Parser.Flag.IEVALUES);
@@ -249,7 +255,7 @@ public class SampleSitesIT {
249255

250256
SiteErrorReporter reporter;
251257

252-
public SampleSitesIT(String uri) throws IOException {
258+
public SampleSitesIT(String uri) throws URISyntaxException, IOException {
253259
super();
254260
agent = new MyDOMUserAgent();
255261

@@ -260,7 +266,7 @@ public SampleSitesIT(String uri) throws IOException {
260266
dom4jAgent = new MyDOM4JUserAgent();
261267

262268
log.info("Testing URL: " + uri);
263-
URL url = new URL(uri);
269+
URL url = new java.net.URI(uri).toURL();
264270

265271
if (errorReporterType == 0) {
266272
reporter = new LogSiteErrorReporter();

ci/io/github/css4j/ci/TreeSiteErrorReporter.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@
1818
import java.io.IOException;
1919
import java.io.OutputStreamWriter;
2020
import java.io.PrintWriter;
21-
import java.net.MalformedURLException;
2221
import java.net.URL;
2322
import java.nio.channels.FileChannel;
2423
import java.nio.charset.StandardCharsets;
@@ -324,10 +323,10 @@ private void selectTargetSheet(StyleSheet sheet, int sheetIndex, boolean warn) {
324323
private String getLinkedSheetInternalPath(StyleSheet sheet, String uri) {
325324
String path;
326325
try {
327-
URL url = new URL(uri);
328-
path = SampleSitesIT.encodeString(url.toExternalForm());
326+
java.net.URI url = new java.net.URI(uri);
327+
path = SampleSitesIT.encodeString(url.toString());
329328
path = url.getHost() + '/' + path;
330-
} catch (MalformedURLException e) {
329+
} catch (Exception e) {
331330
path = "-";
332331
}
333332
return path;

ci/io/github/css4j/ci/ValueComparator.java

Lines changed: 7 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,7 @@
1111

1212
package io.github.css4j.ci;
1313

14-
import java.net.MalformedURLException;
15-
import java.net.URISyntaxException;
16-
import java.net.URL;
14+
import java.net.URI;
1715
import java.util.Iterator;
1816

1917
import org.w3c.dom.DOMException;
@@ -295,17 +293,13 @@ && similarAlphaValue(color.getAlpha(), otherColor.getAlpha())) {
295293
private boolean isSameURI(CSSTypedValue pri, CSSTypedValue primini) {
296294
String uri = pri.getStringValue();
297295
String urimini = primini.getStringValue();
298-
URL baseurl;
296+
URI baseuri;
299297
try {
300-
baseurl = new URL(style.getParentRule().getParentStyleSheet().getHref());
301-
} catch (MalformedURLException e) {
302-
return uri.equals(urimini);
303-
}
304-
try {
305-
URL url = new URL(baseurl, uri);
306-
URL urlmini = new URL(baseurl, urimini);
307-
return url.toURI().normalize().equals(urlmini.toURI().normalize());
308-
} catch (MalformedURLException | URISyntaxException e) {
298+
baseuri = new URI(style.getParentRule().getParentStyleSheet().getHref());
299+
URI u = baseuri.resolve(uri);
300+
URI umini = baseuri.resolve(urimini);
301+
return u.normalize().equals(umini.normalize());
302+
} catch (Exception e) {
309303
}
310304
return uri.equals(urimini);
311305
}

0 commit comments

Comments
 (0)