Polishing Optional usage

quaff · quaff · commit 58bbc0b0406a · 2024-03-11T15:43:51.000+08:00
diff --git a/cdi/src/main/java/org/hibernate/validator/cdi/ValidationExtension.java b/cdi/src/main/java/org/hibernate/validator/cdi/ValidationExtension.java
@@ -280,12 +280,7 @@ private <T> void determineConstrainedMethods(AnnotatedType<T> type, BeanDescript
 			}
 
 			boolean needsValidation;
-			if ( correspondingProperty.isPresent() ) {
-				needsValidation = isGetterConstrained( beanDescriptor, method, correspondingProperty.get() );
-			}
-			else {
-				needsValidation = isNonGetterConstrained( beanDescriptor, method );
-			}
+			needsValidation = correspondingProperty.map( s -> isGetterConstrained( beanDescriptor, method, s ) ).orElseGet( () -> isNonGetterConstrained( beanDescriptor, method ) );
 
 			if ( needsValidation ) {
 				callables.add( annotatedMethod );
diff --git a/engine/src/main/java/org/hibernate/validator/internal/cfg/context/TypeConstraintMappingContextImpl.java b/engine/src/main/java/org/hibernate/validator/internal/cfg/context/TypeConstraintMappingContextImpl.java
@@ -135,7 +135,7 @@ public PropertyConstraintMappingContext field(String property) {
 
 		Optional<JavaBeanField> foundField = javaBeanHelper.findDeclaredField( beanClass, property );
 
-		if ( !foundField.isPresent() ) {
+		if ( foundField.isEmpty() ) {
 			throw LOG.getUnableToFindPropertyWithAccessException( beanClass, property, ElementType.FIELD );
 		}
 
@@ -157,7 +157,7 @@ public PropertyConstraintMappingContext getter(String property) {
 
 		Optional<JavaBeanGetter> foundGetter = javaBeanHelper.findDeclaredGetter( beanClass, property );
 
-		if ( !foundGetter.isPresent() ) {
+		if ( foundGetter.isEmpty() ) {
 			throw LOG.getUnableToFindPropertyWithAccessException( beanClass, property, ElementType.METHOD );
 		}
 
@@ -179,7 +179,7 @@ public MethodConstraintMappingContext method(String name, Class<?>... parameterT
 
 		Optional<JavaBeanMethod> foundMethod = javaBeanHelper.findDeclaredMethod( beanClass, name, parameterTypes );
 
-		if ( !foundMethod.isPresent() ) {
+		if ( foundMethod.isEmpty() ) {
 			throw LOG.getBeanDoesNotContainMethodException( beanClass, name, parameterTypes );
 		}
 
@@ -203,7 +203,7 @@ public MethodConstraintMappingContext method(String name, Class<?>... parameterT
 	public ConstructorConstraintMappingContext constructor(Class<?>... parameterTypes) {
 		Optional<JavaBeanConstructor> foundConstructor = javaBeanHelper.findDeclaredConstructor( beanClass, parameterTypes );
 
-		if ( !foundConstructor.isPresent() ) {
+		if ( foundConstructor.isEmpty() ) {
 			throw LOG.getBeanDoesNotContainConstructorException(
 					beanClass,
 					parameterTypes
diff --git a/engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorImpl.java b/engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorImpl.java
@@ -831,7 +831,7 @@ private <T> void validateParametersInContext(ExecutableValidationContext<T> vali
 
 		Optional<ExecutableMetaData> executableMetaDataOptional = validationContext.getExecutableMetaData();
 
-		if ( !executableMetaDataOptional.isPresent() ) {
+		if ( executableMetaDataOptional.isEmpty() ) {
 			// the method is unconstrained
 			return;
 		}
@@ -1015,7 +1015,7 @@ private <V, T> void validateReturnValueInContext(ExecutableValidationContext<T>
 
 		Optional<ExecutableMetaData> executableMetaDataOptional = validationContext.getExecutableMetaData();
 
-		if ( !executableMetaDataOptional.isPresent() ) {
+		if ( executableMetaDataOptional.isEmpty() ) {
 			// the method is unconstrained
 			return;
 		}
diff --git a/engine/src/main/java/org/hibernate/validator/internal/engine/constraintvalidation/ComposingConstraintTree.java b/engine/src/main/java/org/hibernate/validator/internal/engine/constraintvalidation/ComposingConstraintTree.java
@@ -105,7 +105,7 @@ protected void validateConstraints(ValidationContext<?> validationContext,
 
 			// We re-evaluate the boolean composition by taking into consideration also the violations
 			// from the local constraintValidator
-			if ( !violatedLocalConstraintValidatorContext.isPresent() ) {
+			if ( violatedLocalConstraintValidatorContext.isEmpty() ) {
 				compositionResult.setAtLeastOneTrue( true );
 			}
 			else {
@@ -168,7 +168,7 @@ private void prepareFinalConstraintViolations(ValidationContext<?> validationCon
 			// violations or not (or if there is no local ConstraintValidator at all).
 			// If not we create a violation
 			// using the error message in the annotation declaration at top level.
-			if ( !localConstraintValidatorContext.isPresent() ) {
+			if ( localConstraintValidatorContext.isEmpty() ) {
 				violatedConstraintValidatorContexts.add(
 						validationContext.createConstraintValidatorContextFor(
 								descriptor, valueContext.getPropertyPath()
@@ -185,9 +185,7 @@ private void prepareFinalConstraintViolations(ValidationContext<?> validationCon
 		// as checked in test CustomErrorMessage.java
 		// If no violations have been reported from the local ConstraintValidator, or no such validator exists,
 		// then we just add an empty list.
-		if ( localConstraintValidatorContext.isPresent() ) {
-			violatedConstraintValidatorContexts.add( localConstraintValidatorContext.get() );
-		}
+		localConstraintValidatorContext.ifPresent( violatedConstraintValidatorContexts::add );
 	}
 
 	/**
diff --git a/engine/src/main/java/org/hibernate/validator/internal/engine/validationcontext/ParameterExecutableValidationContext.java b/engine/src/main/java/org/hibernate/validator/internal/engine/validationcontext/ParameterExecutableValidationContext.java
@@ -86,7 +86,7 @@ public Optional<ExecutableMetaData> getExecutableMetaData() {
 	}
 
 	private static boolean buildDisableAlreadyValidatedBeanTracking(Optional<ExecutableMetaData> executableMetaData) {
-		if ( !executableMetaData.isPresent() ) {
+		if ( executableMetaData.isEmpty() ) {
 			// the method is unconstrained so there's no need to worry about the tracking
 			return false;
 		}
diff --git a/engine/src/main/java/org/hibernate/validator/internal/engine/validationcontext/ReturnValueExecutableValidationContext.java b/engine/src/main/java/org/hibernate/validator/internal/engine/validationcontext/ReturnValueExecutableValidationContext.java
@@ -79,7 +79,7 @@ public Optional<ExecutableMetaData> getExecutableMetaData() {
 	}
 
 	private static boolean buildDisableAlreadyValidatedBeanTracking(Optional<ExecutableMetaData> executableMetaData) {
-		if ( !executableMetaData.isPresent() ) {
+		if ( executableMetaData.isEmpty() ) {
 			// the method is unconstrained so there's no need to worry about the tracking
 			return false;
 		}
diff --git a/engine/src/main/java/org/hibernate/validator/internal/engine/valueextraction/OptionalValueExtractor.java b/engine/src/main/java/org/hibernate/validator/internal/engine/valueextraction/OptionalValueExtractor.java
@@ -24,6 +24,6 @@ private OptionalValueExtractor() {
 
 	@Override
 	public void extractValues(Optional<?> originalValue, ValueExtractor.ValueReceiver receiver) {
-		receiver.value( null, originalValue.isPresent() ? originalValue.get() : null );
+		receiver.value( null, originalValue.orElse( null ) );
 	}
 }
diff --git a/engine/src/main/java/org/hibernate/validator/internal/xml/config/ValidationConfigStaxBuilder.java b/engine/src/main/java/org/hibernate/validator/internal/xml/config/ValidationConfigStaxBuilder.java
@@ -274,9 +274,7 @@ protected String getAcceptableQName() {
 		@Override
 		protected void add(XMLEventReader xmlEventReader, XMLEvent xmlEvent) throws XMLStreamException {
 			Optional<String> enabledAttribute = readAttribute( xmlEvent.asStartElement(), ENABLED_QNAME );
-			if ( enabledAttribute.isPresent() ) {
-				enabled = Boolean.parseBoolean( enabledAttribute.get() );
-			}
+			enabledAttribute.ifPresent( s -> enabled = Boolean.parseBoolean( s ) );
 
 			while ( !( xmlEvent.isEndElement() && xmlEvent.asEndElement().getName().getLocalPart().equals( EXECUTABLE_VALIDATION_QNAME_LOCAL_PART ) ) ) {
 				XMLEvent currentEvent = xmlEventReader.nextEvent();
diff --git a/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/AbstractConstrainedElementStaxBuilder.java b/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/AbstractConstrainedElementStaxBuilder.java
@@ -66,7 +66,7 @@ abstract class AbstractConstrainedElementStaxBuilder extends AbstractStaxBuilder
 	protected void add(XMLEventReader xmlEventReader, XMLEvent xmlEvent) throws XMLStreamException {
 		Optional<QName> mainAttributeValueQname = getMainAttributeValueQname();
 		if ( mainAttributeValueQname.isPresent() ) {
-			mainAttributeValue = readAttribute( xmlEvent.asStartElement(), mainAttributeValueQname.get() ).get();
+			mainAttributeValue = readAttribute( xmlEvent.asStartElement(), mainAttributeValueQname.get() ).orElseThrow();
 		}
 		ignoreAnnotations = readAttribute( xmlEvent.asStartElement(), IGNORE_ANNOTATIONS_QNAME ).map( Boolean::parseBoolean );
 		ConstraintTypeStaxBuilder constraintTypeStaxBuilder = getNewConstraintTypeStaxBuilder();
diff --git a/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/AbstractConstrainedExecutableElementStaxBuilder.java b/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/AbstractConstrainedExecutableElementStaxBuilder.java
@@ -57,7 +57,7 @@ abstract class AbstractConstrainedExecutableElementStaxBuilder extends AbstractS
 	protected void add(XMLEventReader xmlEventReader, XMLEvent xmlEvent) throws XMLStreamException {
 		Optional<QName> mainAttributeValueQname = getMainAttributeValueQname();
 		if ( mainAttributeValueQname.isPresent() ) {
-			mainAttributeValue = readAttribute( xmlEvent.asStartElement(), mainAttributeValueQname.get() ).get();
+			mainAttributeValue = readAttribute( xmlEvent.asStartElement(), mainAttributeValueQname.get() ).orElseThrow();
 		}
 		ignoreAnnotations = readAttribute( xmlEvent.asStartElement(), IGNORE_ANNOTATIONS_QNAME ).map( Boolean::parseBoolean );
 		ConstrainedParameterStaxBuilder constrainedParameterStaxBuilder = getNewConstrainedParameterStaxBuilder();
diff --git a/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ClassConstraintTypeStaxBuilder.java b/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ClassConstraintTypeStaxBuilder.java
@@ -102,12 +102,7 @@ ConstrainedType build(Class<?> beanClass) {
 				.collect( Collectors.toSet() );
 
 		// ignore annotation
-		if ( ignoreAnnotations.isPresent() ) {
-			annotationProcessingOptions.ignoreClassLevelConstraintAnnotations(
-					beanClass,
-					ignoreAnnotations.get()
-			);
-		}
+		ignoreAnnotations.ifPresent( b -> annotationProcessingOptions.ignoreClassLevelConstraintAnnotations( beanClass, b ) );
 
 		return new ConstrainedType(
 				ConfigurationSource.XML,
diff --git a/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ConstrainedConstructorStaxBuilder.java b/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ConstrainedConstructorStaxBuilder.java
@@ -76,12 +76,7 @@ ConstrainedExecutable build(JavaBeanHelper javaBeanHelper, Class<?> beanClass, L
 		}
 
 		// ignore annotations
-		if ( ignoreAnnotations.isPresent() ) {
-			annotationProcessingOptions.ignoreConstraintAnnotationsOnMember(
-					javaBeanConstructor,
-					ignoreAnnotations.get()
-			);
-		}
+		ignoreAnnotations.ifPresent( b -> annotationProcessingOptions.ignoreConstraintAnnotationsOnMember( javaBeanConstructor, b ) );
 
 		List<ConstrainedParameter> constrainedParameters = CollectionHelper.newArrayList( constrainedParameterStaxBuilders.size() );
 		for ( int index = 0; index < constrainedParameterStaxBuilders.size(); index++ ) {
diff --git a/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ConstrainedFieldStaxBuilder.java b/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ConstrainedFieldStaxBuilder.java
@@ -82,12 +82,7 @@ ConstrainedField build(JavaBeanHelper javaBeanHelper, Class<?> beanClass, List<S
 		);
 
 		// ignore annotations
-		if ( ignoreAnnotations.isPresent() ) {
-			annotationProcessingOptions.ignoreConstraintAnnotationsOnMember(
-					javaBeanField,
-					ignoreAnnotations.get()
-			);
-		}
+		ignoreAnnotations.ifPresent( b -> annotationProcessingOptions.ignoreConstraintAnnotationsOnMember( javaBeanField, b ) );
 
 		return constrainedField;
 	}
diff --git a/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ConstrainedGetterStaxBuilder.java b/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ConstrainedGetterStaxBuilder.java
@@ -86,12 +86,7 @@ ConstrainedExecutable build(JavaBeanHelper javaBeanHelper, Class<?> beanClass, L
 		);
 
 		// ignore annotations
-		if ( ignoreAnnotations.isPresent() ) {
-			annotationProcessingOptions.ignoreConstraintAnnotationsOnMember(
-					javaBeanGetter,
-					ignoreAnnotations.get()
-			);
-		}
+		ignoreAnnotations.ifPresent( b -> annotationProcessingOptions.ignoreConstraintAnnotationsOnMember( javaBeanGetter, b ) );
 
 		return constrainedGetter;
 	}
diff --git a/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ConstrainedMethodStaxBuilder.java b/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ConstrainedMethodStaxBuilder.java
@@ -79,12 +79,7 @@ ConstrainedExecutable build(JavaBeanHelper javaBeanHelper, Class<?> beanClass, L
 		}
 
 		// ignore annotations
-		if ( ignoreAnnotations.isPresent() ) {
-			annotationProcessingOptions.ignoreConstraintAnnotationsOnMember(
-					javaBeanMethod,
-					ignoreAnnotations.get()
-			);
-		}
+		ignoreAnnotations.ifPresent( b -> annotationProcessingOptions.ignoreConstraintAnnotationsOnMember( javaBeanMethod, b ) );
 
 		List<ConstrainedParameter> constrainedParameters = CollectionHelper.newArrayList( constrainedParameterStaxBuilders.size() );
 		for ( int index = 0; index < constrainedParameterStaxBuilders.size(); index++ ) {
diff --git a/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ConstrainedParameterStaxBuilder.java b/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ConstrainedParameterStaxBuilder.java
@@ -77,13 +77,7 @@ ConstrainedParameter build(Callable callable, int index) {
 		ContainerElementTypeConfiguration containerElementTypeConfiguration = getContainerElementTypeConfiguration( type, constraintLocation );
 
 		// ignore annotations
-		if ( ignoreAnnotations.isPresent() ) {
-			annotationProcessingOptions.ignoreConstraintAnnotationsOnParameter(
-					callable,
-					index,
-					ignoreAnnotations.get()
-			);
-		}
+		ignoreAnnotations.ifPresent( b -> annotationProcessingOptions.ignoreConstraintAnnotationsOnParameter( callable, index, b ) );
 
 		ConstrainedParameter constrainedParameter = new ConstrainedParameter(
 				ConfigurationSource.XML,
diff --git a/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ConstraintTypeStaxBuilder.java b/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ConstraintTypeStaxBuilder.java
@@ -122,9 +122,7 @@ <A extends Annotation> MetaConstraint<A> build(ConstraintLocation constraintLoca
 
 		// set common things to all constraints:
 		Optional<String> message = messageStaxBuilder.build();
-		if ( message.isPresent() ) {
-			annotationDescriptorBuilder.setMessage( message.get() );
-		}
+		message.ifPresent( annotationDescriptorBuilder::setMessage );
 		annotationDescriptorBuilder.setGroups( groupsStaxBuilder.build() )
 				.setPayload( payloadStaxBuilder.build() );
 
diff --git a/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ContainerElementTypeStaxBuilder.java b/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ContainerElementTypeStaxBuilder.java
@@ -79,9 +79,7 @@ protected String getAcceptableQName() {
 	@Override
 	protected void add(XMLEventReader xmlEventReader, XMLEvent xmlEvent) throws XMLStreamException {
 		Optional<String> typeArgumentIndex = readAttribute( xmlEvent.asStartElement(), TYPE_ARGUMENT_INDEX_QNAME );
-		if ( typeArgumentIndex.isPresent() ) {
-			this.typeArgumentIndex = Integer.parseInt( typeArgumentIndex.get() );
-		}
+		typeArgumentIndex.ifPresent( s -> this.typeArgumentIndex = Integer.parseInt( s ) );
 		ConstraintTypeStaxBuilder constraintTypeStaxBuilder = getNewConstraintTypeStaxBuilder();
 		ContainerElementTypeStaxBuilder containerElementTypeConfigurationStaxBuilder = getNewContainerElementTypeConfigurationStaxBuilder();
 		while ( !( xmlEvent.isEndElement() && xmlEvent.asEndElement().getName().getLocalPart().equals( getAcceptableQName() ) ) ) {
diff --git a/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/CrossParameterStaxBuilder.java b/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/CrossParameterStaxBuilder.java
@@ -87,12 +87,7 @@ Set<MetaConstraint<?>> build(Callable callable) {
 				.collect( Collectors.toSet() );
 
 		// ignore annotations
-		if ( ignoreAnnotations.isPresent() ) {
-			annotationProcessingOptions.ignoreConstraintAnnotationsForCrossParameterConstraint(
-					callable,
-					ignoreAnnotations.get()
-			);
-		}
+		ignoreAnnotations.ifPresent( b -> annotationProcessingOptions.ignoreConstraintAnnotationsForCrossParameterConstraint( callable, b ) );
 
 		return crossParameterConstraints;
 	}
diff --git a/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ReturnValueStaxBuilder.java b/engine/src/main/java/org/hibernate/validator/internal/xml/mapping/ReturnValueStaxBuilder.java
@@ -62,12 +62,7 @@ CascadingMetaDataBuilder build(
 		returnValueTypeArgumentConstraints.addAll( containerElementTypeConfiguration.getMetaConstraints() );
 
 		// ignore annotations
-		if ( ignoreAnnotations.isPresent() ) {
-			annotationProcessingOptions.ignoreConstraintAnnotationsForReturnValue(
-					callable,
-					ignoreAnnotations.get()
-			);
-		}
+		ignoreAnnotations.ifPresent( b -> annotationProcessingOptions.ignoreConstraintAnnotationsForReturnValue( callable, b ) );
 
 		return getCascadingMetaData( containerElementTypeConfiguration.getTypeParametersCascadingMetaData(), callable.getType() );
 	}

Original file line number	Diff line number	Diff line change
`@@ -831,7 +831,7 @@ private <T> void validateParametersInContext(ExecutableValidationContext<T> vali`
`831`	`831`
`832`	`832`	`Optional<ExecutableMetaData> executableMetaDataOptional = validationContext.getExecutableMetaData();`
`833`	`833`
`834`		`- if ( !executableMetaDataOptional.isPresent() ) {`
	`834`	`+ if ( executableMetaDataOptional.isEmpty() ) {`
`835`	`835`	`// the method is unconstrained`
`836`	`836`	`return;`
`837`	`837`	`}`
`@@ -1015,7 +1015,7 @@ private <V, T> void validateReturnValueInContext(ExecutableValidationContext<T>`
`1015`	`1015`
`1016`	`1016`	`Optional<ExecutableMetaData> executableMetaDataOptional = validationContext.getExecutableMetaData();`
`1017`	`1017`
`1018`		`- if ( !executableMetaDataOptional.isPresent() ) {`
	`1018`	`+ if ( executableMetaDataOptional.isEmpty() ) {`
`1019`	`1019`	`// the method is unconstrained`
`1020`	`1020`	`return;`
`1021`	`1021`	`}`
Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,7 @@ public Optional<ExecutableMetaData> getExecutableMetaData() {`
`86`	`86`	`}`
`87`	`87`
`88`	`88`	`private static boolean buildDisableAlreadyValidatedBeanTracking(Optional<ExecutableMetaData> executableMetaData) {`
`89`		`- if ( !executableMetaData.isPresent() ) {`
	`89`	`+ if ( executableMetaData.isEmpty() ) {`
`90`	`90`	`// the method is unconstrained so there's no need to worry about the tracking`
`91`	`91`	`return false;`
`92`	`92`	`}`
Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ public Optional<ExecutableMetaData> getExecutableMetaData() {`
`79`	`79`	`}`
`80`	`80`
`81`	`81`	`private static boolean buildDisableAlreadyValidatedBeanTracking(Optional<ExecutableMetaData> executableMetaData) {`
`82`		`- if ( !executableMetaData.isPresent() ) {`
	`82`	`+ if ( executableMetaData.isEmpty() ) {`
`83`	`83`	`// the method is unconstrained so there's no need to worry about the tracking`
`84`	`84`	`return false;`
`85`	`85`	`}`
Original file line number	Diff line number	Diff line change
`@@ -24,6 +24,6 @@ private OptionalValueExtractor() {`
`24`	`24`
`25`	`25`	`@Override`
`26`	`26`	`public void extractValues(Optional<?> originalValue, ValueExtractor.ValueReceiver receiver) {`
`27`		`- receiver.value( null, originalValue.isPresent() ? originalValue.get() : null );`
	`27`	`+ receiver.value( null, originalValue.orElse( null ) );`
`28`	`28`	`}`
`29`	`29`	`}`
Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ abstract class AbstractConstrainedElementStaxBuilder extends AbstractStaxBuilder`
`66`	`66`	`protected void add(XMLEventReader xmlEventReader, XMLEvent xmlEvent) throws XMLStreamException {`
`67`	`67`	`Optional<QName> mainAttributeValueQname = getMainAttributeValueQname();`
`68`	`68`	`if ( mainAttributeValueQname.isPresent() ) {`
`69`		`- mainAttributeValue = readAttribute( xmlEvent.asStartElement(), mainAttributeValueQname.get() ).get();`
	`69`	`+ mainAttributeValue = readAttribute( xmlEvent.asStartElement(), mainAttributeValueQname.get() ).orElseThrow();`
`70`	`70`	`}`
`71`	`71`	`ignoreAnnotations = readAttribute( xmlEvent.asStartElement(), IGNORE_ANNOTATIONS_QNAME ).map( Boolean::parseBoolean );`
`72`	`72`	`ConstraintTypeStaxBuilder constraintTypeStaxBuilder = getNewConstraintTypeStaxBuilder();`
Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ abstract class AbstractConstrainedExecutableElementStaxBuilder extends AbstractS`
`57`	`57`	`protected void add(XMLEventReader xmlEventReader, XMLEvent xmlEvent) throws XMLStreamException {`
`58`	`58`	`Optional<QName> mainAttributeValueQname = getMainAttributeValueQname();`
`59`	`59`	`if ( mainAttributeValueQname.isPresent() ) {`
`60`		`- mainAttributeValue = readAttribute( xmlEvent.asStartElement(), mainAttributeValueQname.get() ).get();`
	`60`	`+ mainAttributeValue = readAttribute( xmlEvent.asStartElement(), mainAttributeValueQname.get() ).orElseThrow();`
`61`	`61`	`}`
`62`	`62`	`ignoreAnnotations = readAttribute( xmlEvent.asStartElement(), IGNORE_ANNOTATIONS_QNAME ).map( Boolean::parseBoolean );`
`63`	`63`	`ConstrainedParameterStaxBuilder constrainedParameterStaxBuilder = getNewConstrainedParameterStaxBuilder();`