Skip to content

Commit 7008f46

Browse files
authored
Merge branch 'openjdk:master' into goetz_backport_8325180
2 parents 974e5f9 + c75eb9d commit 7008f46

File tree

30 files changed

+324
-1580
lines changed

30 files changed

+324
-1580
lines changed

src/hotspot/share/compiler/compileBroker.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2234,7 +2234,7 @@ void CompileBroker::invoke_compiler_on_method(CompileTask* task) {
22342234
}
22352235
}
22362236
}
2237-
if (!task->is_success()) {
2237+
if (!task->is_success() && !JVMCI::in_shutdown()) {
22382238
handle_compile_error(thread, task, nullptr, compilable, failure_reason);
22392239
}
22402240
if (event.should_commit()) {

src/java.base/share/classes/java/text/DateFormatSymbols.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -612,7 +612,8 @@ public void setZoneStrings(String[][] newZoneStrings) {
612612
for (int i = 0; i < newZoneStrings.length; ++i) {
613613
int len = newZoneStrings[i].length;
614614
if (len < 5) {
615-
throw new IllegalArgumentException();
615+
throw new IllegalArgumentException(String.format(
616+
"Row %s of the input array does not have a length of at least 5", i));
616617
}
617618
aCopy[i] = Arrays.copyOf(newZoneStrings[i], len);
618619
}

src/java.xml/share/classes/com/sun/org/apache/xalan/internal/xsltc/compiler/XSLTC.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright (c) 2012, 2022, Oracle and/or its affiliates. All rights reserved.
2+
* Copyright (c) 2012, 2025, Oracle and/or its affiliates. All rights reserved.
33
*/
44
/*
55
* Licensed to the Apache Software Foundation (ASF) under one or more
@@ -57,7 +57,7 @@
5757
* @author G. Todd Miller
5858
* @author Morten Jorgensen
5959
* @author John Howard (johnh@schemasoft.com)
60-
* @LastModified: Jan 2022
60+
* @LastModified: Feb 2025
6161
*/
6262
public final class XSLTC {
6363

@@ -730,7 +730,6 @@ public boolean setDestDirectory(String dstDirName) {
730730
*/
731731
public void setPackageName(String packageName) {
732732
_packageName = Objects.requireNonNull(packageName);
733-
if (_className != null) setClassName(_className);
734733
}
735734

736735
/**

src/java.xml/share/classes/com/sun/org/apache/xalan/internal/xsltc/trax/TransformerFactoryImpl.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright (c) 2007, 2022, Oracle and/or its affiliates. All rights reserved.
2+
* Copyright (c) 2007, 2025, Oracle and/or its affiliates. All rights reserved.
33
*/
44
/*
55
* Licensed to the Apache Software Foundation (ASF) under one or more
@@ -88,7 +88,7 @@
8888
* @author G. Todd Miller
8989
* @author Morten Jorgensen
9090
* @author Santiago Pericas-Geertsen
91-
* @LastModified: Jan 2022
91+
* @LastModified: Feb 2025
9292
*/
9393
public class TransformerFactoryImpl
9494
extends SAXTransformerFactory implements SourceLoader
@@ -1004,9 +1004,6 @@ public Templates newTemplates(Source source)
10041004
// Set the attributes for translet generation
10051005
int outputType = XSLTC.BYTEARRAY_OUTPUT;
10061006
if (_generateTranslet || _autoTranslet) {
1007-
// Set the translet name
1008-
xsltc.setClassName(getTransletBaseName(source));
1009-
10101007
if (_destinationDirectory != null)
10111008
xsltc.setDestDirectory(_destinationDirectory);
10121009
else {
@@ -1020,8 +1017,11 @@ public Templates newTemplates(Source source)
10201017
}
10211018
}
10221019

1020+
// set package name
10231021
if (_packageName != null)
10241022
xsltc.setPackageName(_packageName);
1023+
// Set the translet name
1024+
xsltc.setClassName(getTransletBaseName(source));
10251025

10261026
if (_jarFileName != null) {
10271027
xsltc.setJarFileName(_jarFileName);

test/hotspot/jtreg/serviceability/jvmti/thread/GetStackTrace/GetStackTraceAndRetransformTest/libGetStackTraceAndRetransformTest.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
#include <string.h>
2626
#include "jvmti.h"
2727
#include "jvmti_common.h"
28-
#include "../get_stack_trace.h"
28+
#include "../get_stack_trace.hpp"
2929

3030

3131
extern "C" {

test/hotspot/jtreg/serviceability/jvmti/thread/GetStackTrace/GetStackTraceCurrentThreadTest/libGetStackTraceCurrentThreadTest.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
#include <string.h>
2626
#include "jvmti.h"
2727
#include "jvmti_common.h"
28-
#include "../get_stack_trace.h"
28+
#include "../get_stack_trace.hpp"
2929

3030

3131
extern "C" {

test/hotspot/jtreg/serviceability/jvmti/thread/GetStackTrace/get_stack_trace.h renamed to test/hotspot/jtreg/serviceability/jvmti/thread/GetStackTrace/get_stack_trace.hpp

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright (c) 2003, 2023, Oracle and/or its affiliates. All rights reserved.
2+
* Copyright (c) 2003, 2024, Oracle and/or its affiliates. All rights reserved.
33
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
44
*
55
* This code is free software; you can redistribute it and/or modify it
@@ -21,8 +21,8 @@
2121
* questions.
2222
*/
2323

24-
#ifndef GET_STACK_TRACE_H
25-
#define GET_STACK_TRACE_H
24+
#ifndef GET_STACK_TRACE_HPP
25+
#define GET_STACK_TRACE_HPP
2626
#include "jvmti.h"
2727

2828
typedef struct {
@@ -81,18 +81,18 @@ int compare_stack_trace(jvmtiEnv *jvmti, JNIEnv *jni, jthread thread,
8181
lambda_idx = lambda - expected_frames[exp_idx].cls;
8282
printf("Comparing only first %zu chars in classname.\n", lambda_idx);
8383
}
84-
if (class_signature == NULL || strncmp(class_signature, expected_frames[exp_idx].cls, lambda_idx) != 0) {
84+
if (class_signature == nullptr || strncmp(class_signature, expected_frames[exp_idx].cls, lambda_idx) != 0) {
8585
printf("(frame#%d) wrong class sig: \"%s\", expected: \"%s\"\n",
8686
exp_idx, class_signature, expected_frames[exp_idx].cls);
8787
result = JNI_FALSE;
8888
}
8989

90-
if (name == NULL || strcmp(name, expected_frames[exp_idx].name) != 0) {
90+
if (name == nullptr || strcmp(name, expected_frames[exp_idx].name) != 0) {
9191
printf("(frame#%d) wrong method name: \"%s\", expected: \"%s\"\n",
9292
exp_idx, name, expected_frames[exp_idx].name);
9393
result = JNI_FALSE;
9494
}
95-
if (sig == NULL || strcmp(sig, expected_frames[exp_idx].sig) != 0) {
95+
if (sig == nullptr || strcmp(sig, expected_frames[exp_idx].sig) != 0) {
9696
printf("(frame#%d) wrong method sig: \"%s\", expected: \"%s\"\n",
9797
exp_idx, sig, expected_frames[exp_idx].sig);
9898
result = JNI_FALSE;
@@ -103,4 +103,4 @@ int compare_stack_trace(jvmtiEnv *jvmti, JNIEnv *jni, jthread thread,
103103
}
104104

105105

106-
#endif //GET_STACK_TRACE_H
106+
#endif //GET_STACK_TRACE_HPP

test/hotspot/jtreg/serviceability/jvmti/thread/GetStackTrace/getstacktr03/libgetstacktr03.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
#include <string.h>
2626
#include "jvmti.h"
2727
#include "jvmti_common.h"
28-
#include "../get_stack_trace.h"
28+
#include "../get_stack_trace.hpp"
2929

3030
extern "C" {
3131

test/hotspot/jtreg/serviceability/jvmti/thread/GetStackTrace/getstacktr04/libgetstacktr04.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
#include <string.h>
2626
#include "jvmti.h"
2727
#include "jvmti_common.h"
28-
#include "../get_stack_trace.h"
28+
#include "../get_stack_trace.hpp"
2929

3030
extern "C" {
3131

test/hotspot/jtreg/serviceability/jvmti/thread/GetStackTrace/getstacktr05/libgetstacktr05.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
#include <string.h>
2626
#include "jvmti.h"
2727
#include "jvmti_common.h"
28-
#include "../get_stack_trace.h"
28+
#include "../get_stack_trace.hpp"
2929

3030
extern "C" {
3131

0 commit comments

Comments
 (0)