From e0480f75ac4e0367a053eabd3a07c3fa34eccf61 Mon Sep 17 00:00:00 2001 From: Phillip Webb Date: Wed, 20 Jun 2018 18:45:54 -0700 Subject: [PATCH] Fix javadoc checkstyle issues Fix checkstyle violations for javadoc. Issue: SPR-16968 --- .../aop/MethodBeforeAdvice.java | 5 +- .../aop/aspectj/AbstractAspectJAdvice.java | 12 +- .../AspectJAdviceParameterNameDiscoverer.java | 12 +- .../aop/aspectj/AspectJPointcutAdvisor.java | 4 +- .../aop/aspectj/AspectJProxyUtils.java | 6 +- .../aop/aspectj/DeclareParentsAdvisor.java | 2 +- .../MethodInvocationProceedingJoinPoint.java | 4 +- .../aop/aspectj/RuntimeTestWalker.java | 4 +- .../AbstractAspectJAdvisorFactory.java | 10 +- .../annotation/AspectJProxyFactory.java | 2 +- .../BeanFactoryAspectInstanceFactory.java | 6 +- .../AbstractAdvisingBeanPostProcessor.java | 4 +- .../AbstractSingletonProxyFactoryBean.java | 4 +- .../aop/framework/Advised.java | 4 +- .../aop/framework/AdvisedSupport.java | 14 +-- .../aop/framework/AdvisorChainFactory.java | 4 +- .../aop/framework/AopContext.java | 4 +- .../aop/framework/CglibAopProxy.java | 8 +- .../aop/framework/JdkDynamicAopProxy.java | 8 +- .../aop/framework/ProxyConfig.java | 4 +- .../aop/framework/ProxyCreatorSupport.java | 4 +- .../aop/framework/ProxyFactoryBean.java | 6 +- .../adapter/AdvisorAdapterRegistry.java | 6 +- .../adapter/ThrowsAdviceInterceptor.java | 4 +- .../AbstractAdvisorAutoProxyCreator.java | 2 +- .../autoproxy/AbstractAutoProxyCreator.java | 8 +- .../DefaultAdvisorAutoProxyCreator.java | 4 +- .../autoproxy/ProxyCreationContext.java | 4 +- ...ctBeanFactoryBasedTargetSourceCreator.java | 2 +- .../target/QuickTargetSourceCreator.java | 13 ++- .../interceptor/ExposeBeanNameAdvisors.java | 6 +- .../ExposeInvocationInterceptor.java | 4 +- .../aop/scope/ScopedProxyFactoryBean.java | 8 +- .../aop/support/ComposablePointcut.java | 4 +- .../aop/support/DynamicMethodMatcher.java | 4 +- .../aop/support/Pointcuts.java | 6 +- .../aop/support/RootClassFilter.java | 5 +- .../aop/support/StaticMethodMatcher.java | 4 +- .../AbstractBeanFactoryBasedTargetSource.java | 8 +- .../AbstractLazyCreationTargetSource.java | 6 +- .../target/AbstractPoolingTargetSource.java | 4 +- .../aop/target/CommonsPool2TargetSource.java | 4 +- .../aop/target/EmptyTargetSource.java | 4 +- .../aop/target/HotSwappableTargetSource.java | 6 +- .../aop/target/SingletonTargetSource.java | 6 +- .../AbstractRefreshableTargetSource.java | 4 +- .../SpringConfiguredConfiguration.java | 5 +- .../AbstractNestablePropertyAccessor.java | 10 +- .../org/springframework/beans/BeanUtils.java | 3 +- .../beans/BeanWrapperImpl.java | 2 +- .../beans/CachedIntrospectionResults.java | 8 +- .../beans/ExtendedBeanInfo.java | 12 +- .../beans/MethodInvocationException.java | 4 +- .../beans/MutablePropertyValues.java | 8 +- .../beans/PropertyAccessor.java | 25 +++- .../beans/PropertyBatchUpdateException.java | 4 +- .../beans/PropertyMatches.java | 2 +- .../springframework/beans/PropertyValue.java | 6 +- .../springframework/beans/PropertyValues.java | 4 +- .../BeanNotOfRequiredTypeException.java | 8 +- .../beans/factory/FactoryBean.java | 3 +- .../beans/factory/ObjectFactory.java | 3 +- .../beans/factory/ObjectProvider.java | 3 +- .../beans/factory/SmartFactoryBean.java | 3 +- .../factory/annotation/InjectionMetadata.java | 7 +- .../RequiredAnnotationBeanPostProcessor.java | 2 +- .../factory/config/AbstractFactoryBean.java | 3 +- .../config/AutowireCapableBeanFactory.java | 4 +- .../config/ConfigurableBeanFactory.java | 2 +- .../config/CustomEditorConfigurer.java | 6 +- .../config/FieldRetrievingFactoryBean.java | 7 +- .../config/MethodInvokingFactoryBean.java | 4 +- .../beans/factory/config/NamedBeanHolder.java | 3 +- .../config/PlaceholderConfigurerSupport.java | 22 ++-- .../config/PropertyOverrideConfigurer.java | 7 +- .../beans/factory/config/YamlProcessor.java | 4 +- .../groovy/GroovyBeanDefinitionReader.java | 10 +- .../AbstractAutowireCapableBeanFactory.java | 21 ++-- .../support/AbstractBeanDefinitionReader.java | 4 +- .../factory/support/AbstractBeanFactory.java | 36 +++--- .../support/AutowireCandidateQualifier.java | 5 +- .../support/DefaultListableBeanFactory.java | 28 ++--- .../support/DefaultSingletonBeanRegistry.java | 24 ++-- .../support/FactoryBeanRegistrySupport.java | 2 +- .../beans/factory/support/ManagedArray.java | 4 +- .../beans/factory/support/ManagedList.java | 3 +- .../beans/factory/support/ManagedMap.java | 4 +- .../beans/factory/support/ManagedSet.java | 3 +- .../factory/support/MethodOverrides.java | 4 +- .../PropertiesBeanDefinitionReader.java | 24 ++-- .../factory/support/RootBeanDefinition.java | 22 ++-- .../support/SimpleBeanDefinitionRegistry.java | 4 +- .../support/StaticListableBeanFactory.java | 2 +- .../factory/wiring/BeanConfigurerSupport.java | 2 +- .../xml/AbstractBeanDefinitionParser.java | 6 +- .../xml/BeanDefinitionParserDelegate.java | 4 +- .../xml/DefaultNamespaceHandlerResolver.java | 8 +- .../factory/xml/DelegatingEntityResolver.java | 6 +- .../factory/xml/PluggableSchemaResolver.java | 4 +- .../factory/xml/XmlBeanDefinitionReader.java | 4 +- .../beans/factory/xml/XmlBeanFactory.java | 6 +- .../propertyeditors/CustomBooleanEditor.java | 30 ++++- .../propertyeditors/CustomDateEditor.java | 6 +- .../propertyeditors/CustomNumberEditor.java | 8 +- .../StringArrayPropertyEditor.java | 4 +- .../beans/support/PagedListHolder.java | 9 +- .../beans/support/PropertyComparator.java | 1 + .../context/index/MetadataCollector.java | 6 +- .../cache/ehcache/EhCacheFactoryBean.java | 5 +- .../interceptor/AbstractCacheInterceptor.java | 4 +- .../AbstractJCacheKeyOperation.java | 1 + .../interceptor/AbstractJCacheOperation.java | 7 ++ .../AbstractKeyCacheInterceptor.java | 4 +- .../DefaultCacheInvocationContext.java | 3 +- .../DefaultCacheKeyInvocationContext.java | 3 +- .../DefaultCacheMethodDetails.java | 3 +- .../SimpleExceptionCacheResolver.java | 4 +- .../mail/MailSendException.java | 4 +- .../javamail/ConfigurableMimeFileTypeMap.java | 6 +- .../mail/javamail/JavaMailSenderImpl.java | 6 +- .../mail/javamail/MimeMessageHelper.java | 14 +-- .../quartz/CronTriggerFactoryBean.java | 6 +- .../quartz/JobDetailFactoryBean.java | 4 +- .../quartz/LocalTaskExecutorThreadPool.java | 2 +- .../scheduling/quartz/SchedulerAccessor.java | 4 +- .../quartz/SchedulerFactoryBean.java | 8 +- .../quartz/SimpleTriggerFactoryBean.java | 6 +- .../AnnotationCacheOperationSource.java | 4 +- .../annotation/CacheAnnotationParser.java | 6 +- .../cache/annotation/EnableCaching.java | 31 +++-- ...tationDrivenCacheBeanDefinitionParser.java | 12 +- .../config/CacheManagementConfigUtils.java | 14 ++- .../cache/interceptor/CacheAspectSupport.java | 3 + .../interceptor/CacheEvictOperation.java | 4 +- .../cache/interceptor/CacheOperation.java | 4 +- .../CacheOperationInvocationContext.java | 3 +- .../cache/interceptor/CachePutOperation.java | 4 +- .../cache/interceptor/CacheableOperation.java | 4 +- .../NameMatchCacheOperationSource.java | 4 +- .../interceptor/SimpleCacheResolver.java | 4 +- .../cache/interceptor/SimpleKey.java | 5 +- .../cache/support/NoOpCache.java | 4 +- .../ApplicationContextInitializer.java | 3 +- .../context/ApplicationEvent.java | 6 +- .../ConfigurableApplicationContext.java | 4 +- .../context/EmbeddedValueResolverAware.java | 6 +- .../context/ResourceLoaderAware.java | 4 +- .../context/annotation/AdviceMode.java | 8 +- .../annotation/AdviceModeImportSelector.java | 5 +- .../context/annotation/BeanMethod.java | 6 +- .../context/annotation/ConditionContext.java | 2 +- .../context/annotation/Conditional.java | 4 +- .../context/annotation/Configuration.java | 12 +- .../ConfigurationClassEnhancer.java | 2 +- .../annotation/ConfigurationClassParser.java | 2 +- .../annotation/ConfigurationClassUtils.java | 8 +- .../annotation/ConfigurationCondition.java | 4 +- .../annotation/ConfigurationMethod.java | 4 +- .../annotation/DeferredImportSelector.java | 2 +- .../annotation/EnableLoadTimeWeaving.java | 34 +++--- .../context/annotation/ImportRegistry.java | 6 +- .../annotation/MBeanExportConfiguration.java | 11 +- .../event/SmartApplicationListener.java | 4 +- .../expression/CachedExpressionEvaluator.java | 5 +- .../StandardBeanExpressionResolver.java | 4 +- .../support/AbstractApplicationContext.java | 32 ++--- ...AbstractRefreshableApplicationContext.java | 6 +- .../support/ApplicationObjectSupport.java | 10 +- .../support/ContextTypeMatchClassLoader.java | 4 +- .../support/DefaultLifecycleProcessor.java | 4 +- .../FileSystemXmlApplicationContext.java | 4 +- .../context/support/LiveBeansView.java | 8 +- .../support/MessageSourceAccessor.java | 20 ++-- .../context/support/MessageSourceSupport.java | 4 +- ...ReloadableResourceBundleMessageSource.java | 4 +- .../context/support/StaticMessageSource.java | 4 +- .../weaving/AspectJWeavingEnabler.java | 5 +- ...LocalStatelessSessionProxyFactoryBean.java | 6 +- ...emoteStatelessSessionProxyFactoryBean.java | 6 +- .../format/FormatterRegistry.java | 4 +- .../format/datetime/DateFormatter.java | 4 +- .../DefaultFormattingConversionService.java | 4 +- ...esourceOverridingShadowingClassLoader.java | 4 +- .../classloading/ShadowingClassLoader.java | 4 +- .../jmx/access/MBeanClientInterceptor.java | 4 +- .../access/NotificationListenerRegistrar.java | 4 +- .../jmx/export/MBeanExporter.java | 34 +++--- .../InterfaceBasedMBeanInfoAssembler.java | 6 +- .../SimpleReflectiveMBeanInfoAssembler.java | 8 +- .../jmx/export/metadata/ManagedAttribute.java | 5 +- .../export/naming/IdentityNamingStrategy.java | 8 +- .../ModelMBeanNotificationPublisher.java | 5 +- .../support/ConnectorServerFactoryBean.java | 4 +- .../MBeanServerConnectionFactoryBean.java | 4 +- .../springframework/jndi/JndiCallback.java | 3 +- .../jndi/JndiLocatorSupport.java | 4 +- .../springframework/jndi/JndiTemplate.java | 4 +- .../jndi/support/SimpleJndiBeanFactory.java | 4 +- .../remoting/RemoteAccessException.java | 4 +- .../RemoteInvocationSerializingExporter.java | 4 +- .../remoting/rmi/RmiInvocationWrapper.java | 4 +- .../remoting/support/RemoteInvocation.java | 4 +- .../support/RemoteInvocationResult.java | 4 +- .../remoting/support/RemotingSupport.java | 4 +- .../scheduling/annotation/AsyncResult.java | 3 +- .../scheduling/annotation/EnableAsync.java | 17 ++- .../annotation/EnableScheduling.java | 19 ++- .../annotation/SchedulingConfigurer.java | 4 +- .../config/ScheduledTaskRegistrar.java | 2 +- .../scheduling/support/PeriodicTrigger.java | 6 +- .../support/SimpleTriggerContext.java | 4 +- .../scripting/config/LangNamespaceUtils.java | 4 +- .../config/ScriptingDefaultsParser.java | 4 +- .../support/ResourceScriptSource.java | 4 +- .../support/ScriptFactoryPostProcessor.java | 13 ++- .../springframework/ui/ConcurrentModel.java | 4 +- .../java/org/springframework/ui/Model.java | 6 +- .../java/org/springframework/ui/ModelMap.java | 6 +- .../support/ResourceBundleThemeSource.java | 4 +- .../validation/DataBinder.java | 4 +- .../springframework/validation/Errors.java | 6 +- .../core/AttributeAccessorSupport.java | 2 +- .../org/springframework/core/Constants.java | 8 +- .../core/GenericTypeResolver.java | 4 +- .../springframework/core/MethodParameter.java | 2 +- .../core/NamedInheritableThreadLocal.java | 3 +- .../core/NamedThreadLocal.java | 3 +- .../core/NestedCheckedException.java | 4 +- .../core/NestedRuntimeException.java | 4 +- .../core/OverridingClassLoader.java | 4 +- .../core/ParameterizedTypeReference.java | 1 + .../springframework/core/ResolvableType.java | 17 +-- .../core/SerializableTypeWrapper.java | 12 +- .../core/SimpleAliasRegistry.java | 4 +- .../annotation/AnnotatedElementUtils.java | 23 ++-- .../core/annotation/AnnotationUtils.java | 2 +- .../core/codec/AbstractDataBufferDecoder.java | 1 + .../core/codec/AbstractDecoder.java | 1 + .../core/codec/AbstractEncoder.java | 1 + .../codec/AbstractSingleValueEncoder.java | 5 +- .../core/codec/ByteBufferDecoder.java | 4 +- .../core/codec/ByteBufferEncoder.java | 2 +- .../core/codec/CharSequenceEncoder.java | 3 + .../core/codec/DataBufferDecoder.java | 4 +- .../core/codec/DataBufferEncoder.java | 2 +- .../core/codec/ResourceDecoder.java | 4 +- .../core/codec/ResourceEncoder.java | 5 +- .../core/codec/ResourceRegionEncoder.java | 8 +- .../convert/converter/ConverterFactory.java | 4 +- .../convert/support/ByteBufferConverter.java | 4 +- .../core/env/AbstractPropertyResolver.java | 4 +- .../core/env/CommandLinePropertySource.java | 7 +- .../core/env/ConfigurableEnvironment.java | 10 +- .../core/env/EnumerablePropertySource.java | 3 +- .../core/env/PropertySource.java | 8 +- .../core/env/ReadOnlySystemAttributesMap.java | 4 +- .../core/env/StandardEnvironment.java | 6 +- .../springframework/core/io/PathResource.java | 2 +- .../core/io/ResourceLoader.java | 2 +- .../core/io/buffer/DataBufferFactory.java | 2 +- .../core/io/buffer/DataBufferUtils.java | 8 +- .../core/io/buffer/DefaultDataBuffer.java | 1 + .../core/io/buffer/NettyDataBuffer.java | 2 +- .../io/support/LocalizedResourceHelper.java | 4 +- .../PathMatchingResourcePatternResolver.java | 2 +- .../io/support/PropertiesLoaderSupport.java | 4 +- .../io/support/ResourcePropertySource.java | 4 +- .../core/io/support/ResourceRegion.java | 8 +- .../io/support/SpringFactoriesLoader.java | 4 +- .../core/serializer/Deserializer.java | 3 +- .../core/serializer/Serializer.java | 3 +- .../core/style/ToStringCreator.java | 4 +- .../task/AsyncListenableTaskExecutor.java | 4 +- .../core/task/AsyncTaskExecutor.java | 6 +- .../core/task/SimpleAsyncTaskExecutor.java | 4 +- .../AbstractRecursiveAnnotationVisitor.java | 4 +- .../CachingMetadataReaderFactory.java | 6 +- .../RecursiveAnnotationArrayVisitor.java | 2 + .../RecursiveAnnotationAttributesVisitor.java | 5 +- .../springframework/util/AntPathMatcher.java | 2 +- .../java/org/springframework/util/Assert.java | 28 ++++- .../util/AutoPopulatingList.java | 5 +- .../org/springframework/util/ClassUtils.java | 20 ++-- .../util/CompositeIterator.java | 3 +- .../util/ConcurrencyThrottleSupport.java | 4 +- .../util/ConcurrentReferenceHashMap.java | 16 ++- .../util/FastByteArrayOutputStream.java | 6 +- .../springframework/util/FileCopyUtils.java | 5 +- .../springframework/util/InstanceFilter.java | 3 +- .../util/LinkedCaseInsensitiveMap.java | 1 + .../util/LinkedMultiValueMap.java | 6 +- .../springframework/util/MethodInvoker.java | 4 +- .../org/springframework/util/MimeType.java | 7 +- .../springframework/util/MultiValueMap.java | 4 +- .../springframework/util/ReflectionUtils.java | 4 +- .../springframework/util/ResourceUtils.java | 32 ++--- .../org/springframework/util/StopWatch.java | 6 +- .../org/springframework/util/StreamUtils.java | 5 +- .../util/SystemPropertyUtils.java | 12 +- .../util/comparator/ComparableComparator.java | 5 +- .../util/comparator/CompoundComparator.java | 3 +- .../util/comparator/InstanceComparator.java | 4 +- .../util/comparator/InvertibleComparator.java | 3 +- .../util/comparator/NullSafeComparator.java | 3 +- .../CompletableToListenableFutureAdapter.java | 1 + .../DelegatingCompletableFuture.java | 3 +- .../util/concurrent/ListenableFuture.java | 3 +- .../concurrent/ListenableFutureCallback.java | 3 +- .../ListenableFutureCallbackRegistry.java | 1 + .../util/concurrent/ListenableFutureTask.java | 3 +- .../concurrent/SettableListenableFuture.java | 3 +- .../util/concurrent/SuccessCallback.java | 3 +- .../util/xml/AbstractStaxXMLReader.java | 2 +- .../util/xml/DomContentHandler.java | 4 +- .../springframework/util/xml/DomUtils.java | 4 +- .../util/xml/ListBasedXMLEventReader.java | 4 +- .../springframework/util/xml/StaxSource.java | 4 +- .../springframework/util/xml/StaxUtils.java | 4 +- .../expression/ExpressionException.java | 4 +- .../springframework/expression/Operation.java | 20 +++- .../expression/TypedValue.java | 5 +- .../common/CompositeStringExpression.java | 4 +- .../expression/common/LiteralExpression.java | 4 +- .../expression/spel/CodeFlow.java | 18 ++- .../expression/spel/SpelMessage.java | 3 + .../expression/spel/SpelNode.java | 16 ++- .../spel/ast/CompoundExpression.java | 9 +- .../spel/ast/ConstructorReference.java | 2 +- .../expression/spel/ast/Identifier.java | 5 +- .../expression/spel/ast/Operator.java | 5 +- .../expression/spel/ast/Selection.java | 19 ++- .../expression/spel/ast/TypeCode.java | 27 +++++ .../expression/spel/ast/TypeReference.java | 7 +- .../InternalSpelExpressionParser.java | 2 +- .../spel/support/BooleanTypedValue.java | 10 +- .../spel/support/ReflectionHelper.java | 9 +- .../support/ReflectiveMethodExecutor.java | 4 +- .../support/ReflectivePropertyAccessor.java | 4 + .../support/StandardOperatorOverloader.java | 4 +- .../CannotGetJdbcConnectionException.java | 4 +- ...ffectedIncorrectNumberOfRowsException.java | 8 +- .../jdbc/LobRetrievalFailureException.java | 4 +- .../jdbc/UncategorizedSQLException.java | 4 +- .../config/DatabasePopulatorConfigUtils.java | 2 + .../jdbc/core/BeanPropertyRowMapper.java | 17 +-- .../jdbc/core/CallableStatementCallback.java | 3 +- .../jdbc/core/CallableStatementCreator.java | 4 +- .../core/CallableStatementCreatorFactory.java | 4 +- .../jdbc/core/ConnectionCallback.java | 3 +- .../jdbc/core/JdbcOperations.java | 110 +++++++++--------- .../jdbc/core/JdbcTemplate.java | 34 ++++-- .../jdbc/core/ParameterMapper.java | 6 +- .../ParameterizedPreparedStatementSetter.java | 3 +- .../jdbc/core/PreparedStatementCallback.java | 3 +- .../jdbc/core/PreparedStatementCreator.java | 4 +- .../core/PreparedStatementCreatorFactory.java | 4 +- .../jdbc/core/ResultSetExtractor.java | 5 +- .../core/ResultSetSupportingSqlParameter.java | 20 ++-- .../jdbc/core/RowCountCallbackHandler.java | 10 +- .../springframework/jdbc/core/RowMapper.java | 3 +- .../core/RowMapperResultSetExtractor.java | 3 +- .../jdbc/core/SingleColumnRowMapper.java | 1 + .../jdbc/core/SqlInOutParameter.java | 22 ++-- .../jdbc/core/SqlOutParameter.java | 26 +++-- .../jdbc/core/SqlParameter.java | 12 +- .../jdbc/core/SqlParameterValue.java | 8 +- .../jdbc/core/SqlReturnResultSet.java | 6 +- .../jdbc/core/SqlReturnType.java | 4 +- .../jdbc/core/SqlTypeValue.java | 4 +- .../jdbc/core/StatementCallback.java | 3 +- .../core/metadata/CallMetaDataProvider.java | 4 +- .../metadata/CallMetaDataProviderFactory.java | 4 +- .../metadata/GenericCallMetaDataProvider.java | 2 +- .../GenericTableMetaDataProvider.java | 22 ++-- .../core/metadata/TableMetaDataContext.java | 5 +- .../core/metadata/TableMetaDataProvider.java | 4 +- .../NamedParameterJdbcOperations.java | 62 +++++----- .../NamedParameterJdbcTemplate.java | 6 +- .../core/namedparam/NamedParameterUtils.java | 2 +- .../jdbc/core/simple/AbstractJdbcCall.java | 28 ++--- .../jdbc/core/simple/AbstractJdbcInsert.java | 28 ++--- .../core/simple/SimpleJdbcCallOperations.java | 22 ++-- .../simple/SimpleJdbcInsertOperations.java | 14 +-- ...rruptibleBatchPreparedStatementSetter.java | 4 +- ...bstractLobStreamingResultSetExtractor.java | 3 +- .../core/support/AbstractSqlTypeValue.java | 4 +- .../support/JdbcBeanDefinitionReader.java | 4 +- .../jdbc/core/support/JdbcDaoSupport.java | 4 +- .../jdbc/datasource/AbstractDataSource.java | 4 +- .../jdbc/datasource/ConnectionHolder.java | 5 +- .../DataSourceTransactionManager.java | 4 +- .../datasource/DriverManagerDataSource.java | 4 +- .../IsolationLevelDataSourceAdapter.java | 4 +- .../LazyConnectionDataSourceProxy.java | 4 +- .../datasource/SimpleDriverDataSource.java | 4 +- .../SingleConnectionDataSource.java | 12 +- .../TransactionAwareDataSourceProxy.java | 4 +- .../embedded/EmbeddedDatabaseBuilder.java | 4 +- .../EmbeddedDatabaseConfigurerFactory.java | 2 +- .../embedded/EmbeddedDatabaseFactory.java | 6 +- .../embedded/EmbeddedDatabaseType.java | 8 +- .../jdbc/datasource/init/ScriptUtils.java | 2 +- .../IsolationLevelDataSourceRouter.java | 4 +- .../jdbc/object/BatchSqlUpdate.java | 18 +-- .../jdbc/object/GenericSqlQuery.java | 3 +- .../jdbc/object/MappingSqlQuery.java | 9 +- .../object/MappingSqlQueryWithParameters.java | 11 +- .../jdbc/object/RdbmsOperation.java | 6 +- .../springframework/jdbc/object/SqlCall.java | 6 +- .../jdbc/object/SqlFunction.java | 19 +-- .../jdbc/object/SqlOperation.java | 6 +- .../springframework/jdbc/object/SqlQuery.java | 9 +- .../jdbc/object/SqlUpdate.java | 26 ++--- .../jdbc/object/StoredProcedure.java | 4 +- .../jdbc/object/UpdatableSqlQuery.java | 11 +- ...bstractFallbackSQLExceptionTranslator.java | 6 +- .../support/DatabaseStartupValidator.java | 8 +- .../jdbc/support/JdbcAccessor.java | 2 +- .../jdbc/support/JdbcUtils.java | 2 +- .../SQLErrorCodeSQLExceptionTranslator.java | 10 +- .../jdbc/support/SQLExceptionTranslator.java | 4 +- .../AbstractColumnMaxValueIncrementer.java | 6 +- .../AbstractDataFieldMaxValueIncrementer.java | 6 +- ...ractIdentityColumnMaxValueIncrementer.java | 6 +- .../incrementer/DerbyMaxValueIncrementer.java | 6 +- .../incrementer/MySQLMaxValueIncrementer.java | 10 +- .../rowset/ResultSetWrappingSqlRowSet.java | 6 +- .../ResultSetWrappingSqlRowSetMetaData.java | 4 +- .../jdbc/support/rowset/SqlRowSet.java | 4 +- .../jms/annotation/EnableJms.java | 36 +++--- .../AbstractJmsListenerContainerFactory.java | 1 + .../config/AbstractJmsListenerEndpoint.java | 4 +- .../config/JmsListenerContainerFactory.java | 3 +- .../config/JmsListenerEndpointRegistry.java | 6 +- .../connection/ChainedExceptionListener.java | 2 +- .../connection/SingleConnectionFactory.java | 10 +- .../jms/core/BrowserCallback.java | 3 +- .../jms/core/JmsMessageOperations.java | 4 +- .../springframework/jms/core/JmsTemplate.java | 6 +- .../jms/core/ProducerCallback.java | 3 +- .../jms/core/SessionCallback.java | 3 +- .../jms/core/support/JmsGatewaySupport.java | 4 +- .../listener/SessionAwareMessageListener.java | 3 +- .../AbstractAdaptableMessageListener.java | 4 +- .../jms/listener/adapter/JmsResponse.java | 6 +- .../DefaultJmsActivationSpecFactory.java | 4 +- .../endpoint/JmsActivationSpecConfig.java | 4 +- .../jms/support/JmsAccessor.java | 6 +- .../MappingJackson2MessageConverter.java | 4 +- .../jms/support/converter/MessageType.java | 22 +++- .../converter/MessagingMessageConverter.java | 4 +- .../springframework/messaging/Message.java | 3 +- .../messaging/MessageHeaders.java | 17 ++- ...DestinationResolvingMessagingTemplate.java | 6 +- .../AbstractMessageReceivingTemplate.java | 3 +- .../core/AbstractMessageSendingTemplate.java | 3 +- .../core/AbstractMessagingTemplate.java | 3 +- .../core/CachingDestinationResolverProxy.java | 3 +- .../messaging/core/DestinationResolver.java | 3 +- ...onResolvingMessageReceivingOperations.java | 3 +- ...esolvingMessageRequestReplyOperations.java | 3 +- ...tionResolvingMessageSendingOperations.java | 3 +- .../core/GenericMessagingTemplate.java | 8 +- .../core/MessageReceivingOperations.java | 4 +- .../core/MessageRequestReplyOperations.java | 4 +- .../core/MessageSendingOperations.java | 4 +- .../handler/AbstractMessageCondition.java | 3 +- .../DestinationPatternsMessageCondition.java | 3 + .../messaging/handler/HandlerMethod.java | 4 +- .../DefaultMessageHandlerMethodFactory.java | 4 +- ...inationVariableMethodArgumentResolver.java | 5 +- .../HeadersMethodArgumentResolver.java | 3 +- ...andlerMethodArgumentResolverComposite.java | 6 +- ...dlerMethodReturnValueHandlerComposite.java | 2 +- .../invocation/InvocableHandlerMethod.java | 6 +- .../messaging/simp/SimpAttributes.java | 6 +- .../simp/SimpAttributesContextHolder.java | 8 +- .../simp/SimpMessageMappingInfo.java | 5 +- .../support/MissingSessionUserException.java | 4 +- .../PrincipalMethodArgumentResolver.java | 4 +- .../SimpAnnotationMethodMessageHandler.java | 4 +- .../simp/broker/BrokerAvailabilityEvent.java | 4 +- .../broker/DefaultSubscriptionRegistry.java | 10 +- .../AbstractMessageBrokerConfiguration.java | 4 +- .../simp/stomp/BufferingStompDecoder.java | 2 +- .../simp/stomp/DefaultStompSession.java | 3 + .../stomp/ReactorNettyTcpStompClient.java | 4 +- .../stomp/StompBrokerRelayMessageHandler.java | 5 +- .../messaging/simp/stomp/StompDecoder.java | 8 +- .../messaging/simp/stomp/StompEncoder.java | 4 +- .../support/AbstractHeaderMapper.java | 5 +- .../messaging/support/GenericMessage.java | 3 +- .../messaging/support/MessageBuilder.java | 3 +- .../support/MessageHeaderAccessor.java | 5 +- .../support/NativeMessageHeaderAccessor.java | 3 + .../tcp/FixedIntervalReconnectStrategy.java | 3 +- .../messaging/tcp/TcpConnection.java | 4 +- .../AbstractNioBufferReactorNettyCodec.java | 5 +- .../MonoToListenableFutureAdapter.java | 3 +- .../tcp/reactor/ReactorNettyCodec.java | 5 +- .../tcp/reactor/ReactorNettyTcpClient.java | 1 + .../reactor/ReactorNettyTcpConnection.java | 1 + .../orm/hibernate5/HibernateCallback.java | 3 +- .../HibernateExceptionTranslator.java | 4 +- .../orm/hibernate5/HibernateTemplate.java | 2 +- .../HibernateTransactionManager.java | 6 +- .../hibernate5/LocalSessionFactoryBean.java | 6 +- .../orm/hibernate5/SessionFactoryUtils.java | 2 +- .../support/OpenSessionInViewFilter.java | 5 +- .../jpa/AbstractEntityManagerFactoryBean.java | 10 +- .../orm/jpa/EntityManagerFactoryAccessor.java | 4 +- .../orm/jpa/EntityManagerFactoryUtils.java | 10 +- .../orm/jpa/ExtendedEntityManagerCreator.java | 2 +- .../orm/jpa/JpaTransactionManager.java | 4 +- ...ocalContainerEntityManagerFactoryBean.java | 8 +- .../orm/jpa/SharedEntityManagerCreator.java | 4 +- .../DefaultPersistenceUnitManager.java | 5 +- .../orm/jpa/vendor/HibernateJpaDialect.java | 4 +- .../oxm/castor/CastorMarshaller.java | 4 +- .../oxm/jaxb/Jaxb2Marshaller.java | 4 +- .../oxm/support/AbstractMarshaller.java | 4 +- .../oxm/xstream/CatchAllConverter.java | 4 +- .../oxm/xstream/XStreamMarshaller.java | 50 ++++---- .../mock/jndi/SimpleNamingContextBuilder.java | 4 +- .../mock/web/MockFilterChain.java | 4 +- .../mock/web/MockHttpServletRequest.java | 6 +- .../mock/web/MockHttpSession.java | 3 + .../mock/web/MockServletContext.java | 4 +- .../function/server/MockServerRequest.java | 1 + .../test/annotation/ProfileValueUtils.java | 4 +- .../support/AbstractGenericContextLoader.java | 12 +- .../context/support/DefaultTestContext.java | 2 +- .../support/TestPropertySourceUtils.java | 8 +- .../AbstractTestNGSpringContextTests.java | 4 +- .../TestContextTransactionUtils.java | 2 + .../util/TestContextResourceUtils.java | 3 +- .../web/AbstractGenericWebContextLoader.java | 1 + .../test/util/JsonExpectationsHelper.java | 4 +- .../test/util/MetaAnnotationUtils.java | 4 +- .../test/util/XmlExpectationsHelper.java | 4 +- .../test/util/XpathExpectationsHelper.java | 2 +- .../test/web/ModelAndViewAssert.java | 16 +-- .../SimpleRequestExpectationManager.java | 4 +- .../client/match/ContentRequestMatchers.java | 6 +- .../client/match/JsonPathRequestMatchers.java | 2 +- .../client/match/MockRestRequestMatchers.java | 2 +- .../client/match/XpathRequestMatchers.java | 2 +- .../server/AbstractMockServerSpec.java | 3 +- .../server/DefaultWebTestClientBuilder.java | 10 +- .../reactive/server/FluxExchangeResult.java | 7 +- .../web/reactive/server/HeaderAssertions.java | 2 +- .../web/reactive/server/WebTestClient.java | 25 +++- .../web/reactive/server/WiretapConnector.java | 5 +- .../test/web/servlet/SmartRequestBuilder.java | 6 +- .../htmlunit/DelegatingWebConnection.java | 5 +- .../htmlunit/ForwardRequestPostProcessor.java | 4 +- .../MockMvcWebConnectionBuilderSupport.java | 5 +- .../htmlunit/MockWebResponseBuilder.java | 4 +- .../ConfigurableSmartRequestBuilder.java | 5 +- .../servlet/result/ContentResultMatchers.java | 6 +- .../servlet/result/CookieResultMatchers.java | 4 +- .../servlet/result/HandlerResultMatchers.java | 4 +- .../servlet/result/HeaderResultMatchers.java | 2 +- .../servlet/result/ModelResultMatchers.java | 4 +- .../servlet/result/PrintingResultHandler.java | 5 +- .../servlet/result/StatusResultMatchers.java | 2 +- .../servlet/result/XpathResultMatchers.java | 4 +- .../servlet/setup/AbstractMockMvcBuilder.java | 1 + .../setup/ConfigurableMockMvcBuilder.java | 5 +- .../web/servlet/setup/MockMvcBuilders.java | 2 +- .../setup/StandaloneMockMvcBuilder.java | 4 +- ...ChainedPersistenceExceptionTranslator.java | 2 +- .../dao/support/DaoSupport.java | 4 +- .../dao/support/DataAccessUtils.java | 4 +- .../jca/cci/CannotCreateRecordException.java | 4 +- .../cci/CannotGetCciConnectionException.java | 4 +- .../CciOperationNotSupportedException.java | 4 +- .../cci/RecordTypeNotSupportedException.java | 4 +- .../CciLocalTransactionManager.java | 4 +- .../connection/SingleConnectionFactory.java | 8 +- ...ransactionAwareConnectionFactoryProxy.java | 4 +- .../jca/cci/core/CciTemplate.java | 6 +- .../jca/cci/core/ConnectionCallback.java | 3 +- .../jca/cci/core/InteractionCallback.java | 3 +- .../jca/cci/core/RecordExtractor.java | 3 +- .../jca/cci/core/support/CciDaoSupport.java | 4 +- .../cci/object/MappingCommAreaOperation.java | 4 +- .../cci/object/MappingRecordOperation.java | 4 +- .../jca/cci/object/SimpleRecordOperation.java | 4 +- .../jca/context/BootstrapContextAware.java | 4 +- .../context/SpringContextResourceAdapter.java | 5 +- .../AbstractMessageEndpointFactory.java | 2 +- .../HeuristicCompletionException.java | 16 ++- .../PlatformTransactionManager.java | 4 +- .../AnnotationTransactionAttributeSource.java | 4 +- .../EnableTransactionManagement.java | 25 ++-- .../TransactionAnnotationParser.java | 4 +- ...actFallbackTransactionAttributeSource.java | 2 +- ...MatchAlwaysTransactionAttributeSource.java | 4 +- .../MethodMapTransactionAttributeSource.java | 10 +- .../NameMatchTransactionAttributeSource.java | 4 +- .../RuleBasedTransactionAttribute.java | 8 +- .../interceptor/TransactionAspectSupport.java | 4 +- .../TransactionAttributeSource.java | 4 +- .../TransactionProxyFactoryBean.java | 37 +++--- .../jta/JtaTransactionManager.java | 4 +- .../jta/SimpleTransactionFactory.java | 4 +- .../AbstractPlatformTransactionManager.java | 10 +- .../support/DefaultTransactionDefinition.java | 10 +- .../ResourceHolderSynchronization.java | 4 +- .../support/SimpleTransactionScope.java | 5 +- .../support/TransactionCallback.java | 3 +- .../support/TransactionSynchronization.java | 8 +- .../TransactionSynchronizationUtils.java | 8 +- .../support/TransactionTemplate.java | 2 +- .../http/ContentDisposition.java | 10 +- .../org/springframework/http/HttpEntity.java | 3 +- .../org/springframework/http/HttpHeaders.java | 6 +- .../springframework/http/RequestEntity.java | 3 +- .../springframework/http/ResponseEntity.java | 5 +- ...rceptingAsyncClientHttpRequestFactory.java | 4 +- .../client/InterceptingClientHttpRequest.java | 2 +- .../InterceptingClientHttpRequestFactory.java | 4 +- .../http/client/MultipartBodyBuilder.java | 6 +- .../Netty4ClientHttpRequestFactory.java | 4 +- .../http/client/support/HttpAccessor.java | 4 +- .../http/codec/DecoderHttpMessageReader.java | 3 +- .../http/codec/EncoderHttpMessageWriter.java | 1 + .../http/codec/FormHttpMessageReader.java | 3 + .../http/codec/FormHttpMessageWriter.java | 3 + .../http/codec/HttpMessageDecoder.java | 3 +- .../http/codec/HttpMessageEncoder.java | 3 +- .../http/codec/HttpMessageReader.java | 5 +- .../http/codec/HttpMessageWriter.java | 5 +- .../http/codec/ServerSentEvent.java | 3 +- .../ServerSentEventHttpMessageReader.java | 4 +- .../multipart/MultipartHttpMessageWriter.java | 5 +- .../support/ServerDefaultCodecsImpl.java | 2 + .../http/codec/xml/Jaxb2XmlDecoder.java | 2 +- .../http/codec/xml/XmlEventDecoder.java | 12 +- .../AbstractGenericHttpMessageConverter.java | 1 + .../AbstractHttpMessageConverter.java | 3 +- .../converter/FormHttpMessageConverter.java | 3 + .../GenericHttpMessageConverter.java | 3 +- .../http/converter/HttpMessageConverter.java | 3 +- .../converter/StringHttpMessageConverter.java | 7 +- .../AbstractWireFeedHttpMessageConverter.java | 8 +- .../AbstractJackson2HttpMessageConverter.java | 3 + .../AbstractJsonHttpMessageConverter.java | 3 + .../json/Jackson2ObjectMapperBuilder.java | 4 +- .../json/Jackson2ObjectMapperFactoryBean.java | 4 +- .../ExtensionRegistryInitializer.java | 4 +- .../ProtobufHttpMessageConverter.java | 28 ++++- ...rotobufJsonFormatHttpMessageConverter.java | 4 +- .../AbstractJaxb2HttpMessageConverter.java | 3 +- .../xml/AbstractXmlHttpMessageConverter.java | 3 +- .../Jaxb2CollectionHttpMessageConverter.java | 3 +- .../xml/SourceHttpMessageConverter.java | 3 +- .../AbstractListenerReadPublisher.java | 1 + .../AbstractListenerWriteFlushProcessor.java | 3 +- .../AbstractListenerWriteProcessor.java | 3 +- .../server/reactive/ChannelSendOperator.java | 15 +-- .../server/reactive/ServerHttpRequest.java | 4 +- .../reactive/ServletHttpHandlerAdapter.java | 2 +- .../http/server/reactive/SslInfo.java | 4 +- .../remoting/caucho/HessianExporter.java | 3 + .../AbstractHttpInvokerRequestExecutor.java | 4 +- .../jaxws/SimpleJaxWsServiceExporter.java | 5 +- .../SpringServletContainerInitializer.java | 13 +-- .../web/WebApplicationInitializer.java | 33 +++--- .../ContentNegotiationManagerFactoryBean.java | 6 +- ...thExtensionContentNegotiationStrategy.java | 4 +- ...ServletRequestParameterPropertyValues.java | 10 +- ...sfiedServletRequestParameterException.java | 4 +- .../web/bind/WebDataBinder.java | 2 +- .../web/bind/annotation/RequestPart.java | 4 +- .../web/client/AsyncRestTemplate.java | 4 +- .../ExtractingResponseErrorHandler.java | 4 +- .../client/HttpMessageConverterExtractor.java | 1 + .../web/client/ResponseExtractor.java | 1 + .../web/client/RestTemplate.java | 8 +- .../client/support/RestGatewaySupport.java | 4 +- .../AbstractContextLoaderInitializer.java | 4 +- .../web/context/ContextLoader.java | 14 +-- .../web/context/ContextLoaderListener.java | 4 +- .../web/context/ServletConfigAware.java | 4 +- .../web/context/ServletContextAware.java | 4 +- .../request/AbstractRequestAttributes.java | 4 +- .../request/FacesRequestAttributes.java | 4 +- .../context/request/ServletWebRequest.java | 6 +- .../web/context/request/WebRequest.java | 4 +- .../context/request/async/DeferredResult.java | 3 +- .../async/DeferredResultInterceptorChain.java | 5 +- .../async/StandardServletAsyncWebRequest.java | 4 +- .../request/async/WebAsyncManager.java | 2 +- .../context/request/async/WebAsyncTask.java | 3 +- .../context/request/async/WebAsyncUtils.java | 5 +- ...tractRefreshableWebApplicationContext.java | 10 +- .../support/GenericWebApplicationContext.java | 4 +- .../support/GroovyWebApplicationContext.java | 8 +- .../context/support/RequestHandledEvent.java | 10 +- .../support/ServletRequestHandledEvent.java | 12 +- .../support/StandardServletEnvironment.java | 8 +- .../support/WebApplicationContextUtils.java | 10 +- .../support/XmlWebApplicationContext.java | 8 +- .../filter/AbstractRequestLoggingFilter.java | 16 +++ .../web/filter/CharacterEncodingFilter.java | 4 +- .../web/filter/GenericFilterBean.java | 6 +- .../web/filter/HiddenHttpMethodFilter.java | 2 +- .../reactive/HiddenHttpMethodFilter.java | 5 +- .../jsf/DelegatingNavigationHandlerProxy.java | 4 +- .../WebApplicationContextFacesELResolver.java | 2 +- .../web/method/ControllerAdviceBean.java | 2 +- .../web/method/HandlerMethod.java | 4 +- .../web/method/HandlerTypePredicate.java | 3 + ...ractCookieValueMethodArgumentResolver.java | 3 +- ...tractNamedValueMethodArgumentResolver.java | 3 +- ...ExpressionValueMethodArgumentResolver.java | 3 +- .../web/method/annotation/ModelFactory.java | 3 +- .../RequestHeaderMethodArgumentResolver.java | 3 +- .../RequestParamMethodArgumentResolver.java | 4 +- .../CompositeUriComponentsContributor.java | 18 +-- ...andlerMethodArgumentResolverComposite.java | 10 +- ...dlerMethodReturnValueHandlerComposite.java | 6 +- .../support/InvocableHandlerMethod.java | 6 +- .../method/support/ModelAndViewContainer.java | 6 +- .../commons/CommonsFileUploadSupport.java | 4 +- .../multipart/support/MultipartFilter.java | 5 +- .../support/MultipartResolutionDelegate.java | 3 + .../handler/ExceptionHandlingWebHandler.java | 4 +- .../server/handler/FilteringWebHandler.java | 2 +- .../ResponseStatusExceptionHandler.java | 2 +- .../session/DefaultWebSessionManager.java | 2 +- .../session/InMemoryWebSessionStore.java | 2 +- .../web/util/NestedServletException.java | 4 +- .../web/util/ServletContextPropertyUtils.java | 6 +- .../springframework/web/util/TagUtils.java | 10 +- .../web/util/UriComponents.java | 13 ++- .../web/util/UriComponentsBuilder.java | 2 +- .../web/util/UrlPathHelper.java | 6 +- .../springframework/web/util/WebUtils.java | 108 ++++++++++++++--- .../pattern/CaptureTheRestPathElement.java | 5 +- .../pattern/CaptureVariablePathElement.java | 5 +- .../pattern/InternalPathPatternParser.java | 17 +-- .../web/util/pattern/PathElement.java | 8 +- .../web/util/pattern/PathPattern.java | 54 +++++---- .../web/util/pattern/package-info.java | 2 +- .../config/ResourceHandlerRegistry.java | 2 +- .../config/WebFluxConfigurationSupport.java | 2 +- .../reactive/config/WebFluxConfigurer.java | 4 +- .../web/reactive/function/BodyExtractor.java | 4 +- .../web/reactive/function/BodyExtractors.java | 2 +- .../web/reactive/function/BodyInserter.java | 4 +- .../web/reactive/function/BodyInserters.java | 2 + .../function/client/ExchangeStrategies.java | 4 +- .../reactive/function/client/WebClient.java | 28 ++++- .../server/DefaultEntityResponseBuilder.java | 1 + .../server/DefaultServerResponseBuilder.java | 3 + .../function/server/EntityResponse.java | 3 + .../function/server/HandlerStrategies.java | 12 +- .../function/server/RenderingResponse.java | 4 +- .../function/server/RequestPredicates.java | 10 +- .../function/server/RouterFunctions.java | 19 +++ .../function/server/ServerResponse.java | 8 +- .../support/HandlerFunctionAdapter.java | 4 +- .../server/support/RouterFunctionMapping.java | 2 +- .../support/ServerResponseResultHandler.java | 2 +- .../handler/AbstractUrlHandlerMapping.java | 2 +- .../handler/SimpleUrlHandlerMapping.java | 4 +- .../resource/CachingResourceResolver.java | 6 + .../resource/CssLinkResourceTransformer.java | 3 + .../resource/EncodedResourceResolver.java | 8 +- .../resource/GzipResourceResolver.java | 5 +- .../resource/ResourceResolverChain.java | 4 +- .../resource/ResourceTransformerChain.java | 4 +- .../reactive/resource/ResourceWebHandler.java | 4 +- .../resource/VersionResourceResolver.java | 2 +- .../AbstractNameValueExpression.java | 3 +- .../condition/AbstractRequestCondition.java | 4 +- .../result/condition/NameValueExpression.java | 3 +- .../result/condition/RequestCondition.java | 7 +- .../RequestMethodsRequestCondition.java | 6 +- .../method/AbstractHandlerMethodMapping.java | 2 +- .../result/method/InvocableHandlerMethod.java | 2 +- .../result/method/RequestMappingInfo.java | 4 +- .../RequestMappingInfoHandlerMapping.java | 3 +- .../method/SyncInvocableHandlerMethod.java | 2 +- .../AbstractMessageWriterResultHandler.java | 4 +- .../AbstractNamedValueArgumentResolver.java | 9 +- ...bstractNamedValueSyncArgumentResolver.java | 5 +- .../annotation/ControllerMethodResolver.java | 3 +- .../CookieValueMethodArgumentResolver.java | 7 +- ...ExpressionValueMethodArgumentResolver.java | 7 +- .../PathVariableMethodArgumentResolver.java | 7 +- ...equestAttributeMethodArgumentResolver.java | 7 +- .../RequestHeaderMethodArgumentResolver.java | 7 +- .../reactive/result/view/AbstractView.java | 6 +- .../reactive/result/view/RedirectView.java | 2 +- .../web/reactive/result/view/Rendering.java | 4 +- .../reactive/result/view/RequestContext.java | 16 +-- .../web/reactive/result/view/View.java | 2 +- .../result/view/ViewResolverSupport.java | 3 + .../view/script/ScriptTemplateConfigurer.java | 4 +- .../AbstractListenerWebSocketSession.java | 12 +- .../adapter/AbstractWebSocketSession.java | 1 + .../adapter/NettyWebSocketSessionSupport.java | 7 +- .../client/UndertowWebSocketClient.java | 9 +- .../web/servlet/DispatcherServlet.java | 30 ++--- .../web/servlet/FrameworkServlet.java | 34 +++--- .../web/servlet/HandlerAdapter.java | 4 +- .../web/servlet/HttpServletBean.java | 6 +- .../web/servlet/ModelAndView.java | 26 ++--- .../ModelAndViewDefiningException.java | 4 +- .../org/springframework/web/servlet/View.java | 4 +- .../web/servlet/ViewResolver.java | 4 +- .../AnnotationDrivenBeanDefinitionParser.java | 12 +- ...eMarkerConfigurerBeanDefinitionParser.java | 7 +- ...yMarkupConfigurerBeanDefinitionParser.java | 9 +- .../config/ResourcesBeanDefinitionParser.java | 2 +- ...emplateConfigurerBeanDefinitionParser.java | 7 +- .../TilesConfigurerBeanDefinitionParser.java | 9 +- .../ViewResolversBeanDefinitionParser.java | 5 +- .../ContentNegotiationConfigurer.java | 7 +- .../annotation/InterceptorRegistry.java | 6 +- .../annotation/ResourceHandlerRegistry.java | 2 +- .../WebMvcConfigurationSupport.java | 34 +++--- .../config/annotation/WebMvcConfigurer.java | 6 +- .../AbstractHandlerExceptionResolver.java | 2 +- .../handler/AbstractHandlerMapping.java | 14 +-- .../handler/AbstractHandlerMethodMapping.java | 2 +- .../handler/AbstractUrlHandlerMapping.java | 2 +- .../HandlerExceptionResolverComposite.java | 4 +- .../HandlerMethodMappingNamingStrategy.java | 3 +- .../SimpleMappingExceptionResolver.java | 2 +- .../handler/SimpleUrlHandlerMapping.java | 4 +- .../web/servlet/mvc/AbstractController.java | 6 +- .../mvc/UrlFilenameViewController.java | 4 +- .../servlet/mvc/WebContentInterceptor.java | 8 +- .../AbstractNameValueExpression.java | 3 +- .../condition/AbstractRequestCondition.java | 4 +- .../mvc/condition/NameValueExpression.java | 3 +- .../RequestMethodsRequestCondition.java | 6 +- .../method/AbstractHandlerMethodAdapter.java | 2 +- .../mvc/method/RequestMappingInfo.java | 4 +- .../RequestMappingInfoHandlerMapping.java | 3 +- ...nfoHandlerMethodMappingNamingStrategy.java | 4 +- ...essageConverterMethodArgumentResolver.java | 2 +- ...stractMessageConverterMethodProcessor.java | 7 +- .../annotation/MvcUriComponentsBuilder.java | 21 +++- .../annotation/ReactiveTypeHandler.java | 13 ++- .../RequestMappingHandlerAdapter.java | 6 +- .../RequestPartMethodArgumentResolver.java | 8 +- .../method/annotation/ResponseBodyAdvice.java | 3 +- .../annotation/ResponseBodyEmitter.java | 6 +- .../ServletInvocableHandlerMethod.java | 4 +- .../ServletRequestMethodArgumentResolver.java | 5 +- ...ServletResponseMethodArgumentResolver.java | 5 +- .../resource/CachingResourceResolver.java | 6 + .../resource/CssLinkResourceTransformer.java | 5 +- .../DefaultServletHttpRequestHandler.java | 12 +- .../resource/EncodedResourceResolver.java | 8 +- .../resource/GzipResourceResolver.java | 5 +- .../resource/ResourceHttpRequestHandler.java | 4 +- .../resource/ResourceResolverChain.java | 4 +- .../resource/ResourceTransformerChain.java | 4 +- .../resource/VersionResourceResolver.java | 2 +- .../web/servlet/support/RequestContext.java | 22 ++-- .../servlet/support/WebContentGenerator.java | 12 +- .../web/servlet/tags/ArgumentTag.java | 22 ++-- .../web/servlet/tags/BindErrorsTag.java | 33 +++--- .../web/servlet/tags/EscapeBodyTag.java | 31 ++--- .../web/servlet/tags/EvalTag.java | 63 +++++----- .../web/servlet/tags/HtmlEscapeTag.java | 20 ++-- .../web/servlet/tags/MessageTag.java | 104 ++++++++--------- .../web/servlet/tags/NestedPathTag.java | 22 ++-- .../web/servlet/tags/ParamTag.java | 32 ++--- .../servlet/tags/RequestContextAwareTag.java | 4 +- .../web/servlet/tags/ThemeTag.java | 102 ++++++++-------- .../web/servlet/tags/TransformTag.java | 56 ++++----- .../web/servlet/tags/UrlTag.java | 72 ++++++------ .../tags/form/AbstractHtmlElementBodyTag.java | 4 +- .../web/servlet/tags/form/ButtonTag.java | 4 +- .../web/servlet/tags/form/FormTag.java | 8 +- .../servlet/theme/CookieThemeResolver.java | 8 +- .../view/AbstractCachingViewResolver.java | 14 +-- .../web/servlet/view/AbstractView.java | 8 +- .../view/ContentNegotiatingViewResolver.java | 2 +- .../web/servlet/view/RedirectView.java | 8 +- .../view/ResourceBundleViewResolver.java | 2 +- .../servlet/view/UrlBasedViewResolver.java | 8 +- .../web/servlet/view/XmlViewResolver.java | 2 +- .../view/feed/AbstractAtomFeedView.java | 5 +- .../servlet/view/feed/AbstractFeedView.java | 3 +- .../view/freemarker/FreeMarkerView.java | 4 +- .../view/script/ScriptTemplateConfigurer.java | 4 +- .../view/script/ScriptTemplateView.java | 5 +- .../tiles3/SimpleSpringPreparerFactory.java | 4 +- .../view/xml/MappingJackson2XmlView.java | 5 +- .../web/socket/AbstractWebSocketMessage.java | 3 +- .../web/socket/WebSocketMessage.java | 3 +- .../adapter/AbstractWebSocketSession.java | 3 +- ...ebSocketToJettyExtensionConfigAdapter.java | 3 + .../ConvertingEncoderDecoderSupport.java | 12 +- .../client/AbstractWebSocketClient.java | 4 +- .../client/jetty/JettyWebSocketClient.java | 4 +- .../config/HandlersBeanDefinitionParser.java | 4 +- .../MessageBrokerBeanDefinitionParser.java | 4 +- .../AbstractWebSocketHandlerRegistration.java | 3 +- .../config/annotation/EnableWebSocket.java | 4 +- .../ServletWebSocketHandlerRegistry.java | 4 +- .../annotation/SockJsServiceRegistration.java | 3 +- .../annotation/WebSocketConfigurer.java | 4 +- .../WebSocketHandlerRegistration.java | 2 +- .../handler/BeanCreatingHandlerProvider.java | 3 +- .../messaging/StompSubProtocolHandler.java | 6 +- .../messaging/SubProtocolErrorHandler.java | 3 +- .../socket/messaging/SubProtocolHandler.java | 4 +- .../messaging/WebSocketStompClient.java | 14 +-- .../support/OriginHandshakeInterceptor.java | 3 +- .../web/socket/sockjs/SockJsService.java | 4 +- .../socket/sockjs/client/SockJsClient.java | 4 +- .../sockjs/client/UndertowXhrTransport.java | 2 +- .../web/socket/sockjs/frame/SockJsFrame.java | 5 +- .../sockjs/support/AbstractSockJsService.java | 2 + .../transport/SockJsSessionFactory.java | 4 +- .../TransportHandlingSockJsService.java | 2 +- .../handler/HtmlFileTransportHandler.java | 10 +- .../session/AbstractSockJsSession.java | 4 +- 928 files changed, 3729 insertions(+), 2686 deletions(-) diff --git a/spring-aop/src/main/java/org/springframework/aop/MethodBeforeAdvice.java b/spring-aop/src/main/java/org/springframework/aop/MethodBeforeAdvice.java index cb6f8da209a..a390228202e 100644 --- a/spring-aop/src/main/java/org/springframework/aop/MethodBeforeAdvice.java +++ b/spring-aop/src/main/java/org/springframework/aop/MethodBeforeAdvice.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2012 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -24,10 +24,9 @@ import org.springframework.lang.Nullable; * Advice invoked before a method is invoked. Such advices cannot * prevent the method call proceeding, unless they throw a Throwable. * + * @author Rod Johnson * @see AfterReturningAdvice * @see ThrowsAdvice - * - * @author Rod Johnson */ public interface MethodBeforeAdvice extends BeforeAdvice { diff --git a/spring-aop/src/main/java/org/springframework/aop/aspectj/AbstractAspectJAdvice.java b/spring-aop/src/main/java/org/springframework/aop/aspectj/AbstractAspectJAdvice.java index 37e9a596a8d..8951abcd587 100644 --- a/spring-aop/src/main/java/org/springframework/aop/aspectj/AbstractAspectJAdvice.java +++ b/spring-aop/src/main/java/org/springframework/aop/aspectj/AbstractAspectJAdvice.java @@ -117,16 +117,16 @@ public abstract class AbstractAspectJAdvice implements Advice, AspectJPrecedence /** * This will be non-null if the creator of this advice object knows the argument names - * and sets them explicitly + * and sets them explicitly. */ @Nullable private String[] argumentNames; - /** Non-null if after throwing advice binds the thrown value */ + /** Non-null if after throwing advice binds the thrown value. */ @Nullable private String throwingName; - /** Non-null if after returning advice binds the return value */ + /** Non-null if after returning advice binds the return value. */ @Nullable private String returningName; @@ -136,13 +136,13 @@ public abstract class AbstractAspectJAdvice implements Advice, AspectJPrecedence /** * Index for thisJoinPoint argument (currently only - * supported at index 0 if present at all) + * supported at index 0 if present at all). */ private int joinPointArgumentIndex = -1; /** * Index for thisJoinPointStaticPart argument (currently only - * supported at index 0 if present at all) + * supported at index 0 if present at all). */ private int joinPointStaticPartArgumentIndex = -1; @@ -551,7 +551,7 @@ public abstract class AbstractAspectJAdvice implements Advice, AspectJPrecedence /** * Take the arguments at the method execution join point and output a set of arguments - * to the advice method + * to the advice method. * @param jp the current JoinPoint * @param jpMatch the join point match that matched this execution join point * @param returnValue the return value from the method execution (may be null) diff --git a/spring-aop/src/main/java/org/springframework/aop/aspectj/AspectJAdviceParameterNameDiscoverer.java b/spring-aop/src/main/java/org/springframework/aop/aspectj/AspectJAdviceParameterNameDiscoverer.java index d1a9c25c94a..73eea3bd4ef 100644 --- a/spring-aop/src/main/java/org/springframework/aop/aspectj/AspectJAdviceParameterNameDiscoverer.java +++ b/spring-aop/src/main/java/org/springframework/aop/aspectj/AspectJAdviceParameterNameDiscoverer.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -156,17 +156,17 @@ public class AspectJAdviceParameterNameDiscoverer implements ParameterNameDiscov } - /** The pointcut expression associated with the advice, as a simple String */ + /** The pointcut expression associated with the advice, as a simple String. */ @Nullable private String pointcutExpression; private boolean raiseExceptions; - /** If the advice is afterReturning, and binds the return value, this is the parameter name used */ + /** If the advice is afterReturning, and binds the return value, this is the parameter name used. */ @Nullable private String returningName; - /** If the advice is afterThrowing, and binds the thrown value, this is the parameter name used */ + /** If the advice is afterThrowing, and binds the thrown value, this is the parameter name used. */ @Nullable private String throwingName; @@ -178,7 +178,7 @@ public class AspectJAdviceParameterNameDiscoverer implements ParameterNameDiscov /** - * Create a new discoverer that attempts to discover parameter names + * Create a new discoverer that attempts to discover parameter names. * from the given pointcut expression. */ public AspectJAdviceParameterNameDiscoverer(@Nullable String pointcutExpression) { @@ -677,7 +677,7 @@ public class AspectJAdviceParameterNameDiscoverer implements ParameterNameDiscov } /** - * Match up args against unbound arguments of primitive types + * Match up args against unbound arguments of primitive types. */ private void maybeBindPrimitiveArgsFromPointcutExpression() { int numUnboundPrimitives = countNumberOfUnboundPrimitiveArguments(); diff --git a/spring-aop/src/main/java/org/springframework/aop/aspectj/AspectJPointcutAdvisor.java b/spring-aop/src/main/java/org/springframework/aop/aspectj/AspectJPointcutAdvisor.java index b4adec46c17..108c0eeb881 100644 --- a/spring-aop/src/main/java/org/springframework/aop/aspectj/AspectJPointcutAdvisor.java +++ b/spring-aop/src/main/java/org/springframework/aop/aspectj/AspectJPointcutAdvisor.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -43,7 +43,7 @@ public class AspectJPointcutAdvisor implements PointcutAdvisor, Ordered { /** - * Create a new AspectJPointcutAdvisor for the given advice + * Create a new AspectJPointcutAdvisor for the given advice. * @param advice the AbstractAspectJAdvice to wrap */ public AspectJPointcutAdvisor(AbstractAspectJAdvice advice) { diff --git a/spring-aop/src/main/java/org/springframework/aop/aspectj/AspectJProxyUtils.java b/spring-aop/src/main/java/org/springframework/aop/aspectj/AspectJProxyUtils.java index 0fe05596793..6e9bb0563a6 100644 --- a/spring-aop/src/main/java/org/springframework/aop/aspectj/AspectJProxyUtils.java +++ b/spring-aop/src/main/java/org/springframework/aop/aspectj/AspectJProxyUtils.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2012 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -36,8 +36,8 @@ public abstract class AspectJProxyUtils { * This will expose the current Spring AOP invocation (necessary for some AspectJ pointcut matching) * and make available the current AspectJ JoinPoint. The call will have no effect if there are no * AspectJ advisors in the advisor chain. - * @param advisors Advisors available - * @return {@code true} if any special {@link Advisor Advisors} were added, otherwise {@code false}. + * @param advisors the advisors available + * @return {@code true} if any special {@link Advisor Advisors} were added, otherwise {@code false} */ public static boolean makeAdvisorChainAspectJCapableIfNecessary(List advisors) { // Don't add advisors to an empty list; may indicate that proxying is just not required diff --git a/spring-aop/src/main/java/org/springframework/aop/aspectj/DeclareParentsAdvisor.java b/spring-aop/src/main/java/org/springframework/aop/aspectj/DeclareParentsAdvisor.java index d2a607f54e2..0e4ae214733 100644 --- a/spring-aop/src/main/java/org/springframework/aop/aspectj/DeclareParentsAdvisor.java +++ b/spring-aop/src/main/java/org/springframework/aop/aspectj/DeclareParentsAdvisor.java @@ -65,7 +65,7 @@ public class DeclareParentsAdvisor implements IntroductionAdvisor { /** * Private constructor to share common code between impl-based delegate and reference-based delegate - * (cannot use method such as init() to share common code, due the use of final fields) + * (cannot use method such as init() to share common code, due the use of final fields). * @param interfaceType static field defining the introduction * @param typePattern type pattern the introduction is restricted to * @param interceptor the delegation advice as {@link IntroductionInterceptor} diff --git a/spring-aop/src/main/java/org/springframework/aop/aspectj/MethodInvocationProceedingJoinPoint.java b/spring-aop/src/main/java/org/springframework/aop/aspectj/MethodInvocationProceedingJoinPoint.java index 23d3438e2cd..e31b0129dcf 100644 --- a/spring-aop/src/main/java/org/springframework/aop/aspectj/MethodInvocationProceedingJoinPoint.java +++ b/spring-aop/src/main/java/org/springframework/aop/aspectj/MethodInvocationProceedingJoinPoint.java @@ -58,11 +58,11 @@ public class MethodInvocationProceedingJoinPoint implements ProceedingJoinPoint, @Nullable private Object[] args; - /** Lazily initialized signature object */ + /** Lazily initialized signature object. */ @Nullable private Signature signature; - /** Lazily initialized source location object */ + /** Lazily initialized source location object. */ @Nullable private SourceLocation sourceLocation; diff --git a/spring-aop/src/main/java/org/springframework/aop/aspectj/RuntimeTestWalker.java b/spring-aop/src/main/java/org/springframework/aop/aspectj/RuntimeTestWalker.java index 450c4b9201d..59d243764e5 100644 --- a/spring-aop/src/main/java/org/springframework/aop/aspectj/RuntimeTestWalker.java +++ b/spring-aop/src/main/java/org/springframework/aop/aspectj/RuntimeTestWalker.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -51,7 +51,7 @@ import org.springframework.util.ReflectionUtils; * migrate to {@code ShadowMatch.getVariablesInvolvedInRuntimeTest()} * or some similar operation. * - *

See Bug 151593 + *

See Bug 151593 * * @author Adrian Colyer * @author Ramnivas Laddad diff --git a/spring-aop/src/main/java/org/springframework/aop/aspectj/annotation/AbstractAspectJAdvisorFactory.java b/spring-aop/src/main/java/org/springframework/aop/aspectj/annotation/AbstractAspectJAdvisorFactory.java index 70b928ab2fb..0fa9249ae4c 100644 --- a/spring-aop/src/main/java/org/springframework/aop/aspectj/annotation/AbstractAspectJAdvisorFactory.java +++ b/spring-aop/src/main/java/org/springframework/aop/aspectj/annotation/AbstractAspectJAdvisorFactory.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -61,7 +61,7 @@ public abstract class AbstractAspectJAdvisorFactory implements AspectJAdvisorFac private static final String AJC_MAGIC = "ajc$"; - /** Logger available to subclasses */ + /** Logger available to subclasses. */ protected final Log logger = LogFactory.getLog(getClass()); protected final ParameterNameDiscoverer parameterNameDiscoverer = new AspectJAnnotationParameterNameDiscoverer(); @@ -123,7 +123,7 @@ public abstract class AbstractAspectJAdvisorFactory implements AspectJAdvisorFac /** * Find and return the first AspectJ annotation on the given method - * (there should only be one anyway...) + * (there should only be one anyway...). */ @SuppressWarnings("unchecked") @Nullable @@ -151,6 +151,9 @@ public abstract class AbstractAspectJAdvisorFactory implements AspectJAdvisorFac } + /** + * AspectJ annotation types. + */ protected enum AspectJAnnotationType { AtPointcut, @@ -165,6 +168,7 @@ public abstract class AbstractAspectJAdvisorFactory implements AspectJAdvisorFac /** * Class modelling an AspectJ annotation, exposing its type enumeration and * pointcut String. + * @param the annotation type */ protected static class AspectJAnnotation { diff --git a/spring-aop/src/main/java/org/springframework/aop/aspectj/annotation/AspectJProxyFactory.java b/spring-aop/src/main/java/org/springframework/aop/aspectj/annotation/AspectJProxyFactory.java index 5c4fe918c1f..f1b7c86c58c 100644 --- a/spring-aop/src/main/java/org/springframework/aop/aspectj/annotation/AspectJProxyFactory.java +++ b/spring-aop/src/main/java/org/springframework/aop/aspectj/annotation/AspectJProxyFactory.java @@ -49,7 +49,7 @@ import org.springframework.util.ClassUtils; @SuppressWarnings("serial") public class AspectJProxyFactory extends ProxyCreatorSupport { - /** Cache for singleton aspect instances */ + /** Cache for singleton aspect instances. */ private static final Map, Object> aspectCache = new ConcurrentHashMap<>(); private final AspectJAdvisorFactory aspectFactory = new ReflectiveAspectJAdvisorFactory(); diff --git a/spring-aop/src/main/java/org/springframework/aop/aspectj/annotation/BeanFactoryAspectInstanceFactory.java b/spring-aop/src/main/java/org/springframework/aop/aspectj/annotation/BeanFactoryAspectInstanceFactory.java index ccd581cf7b6..35edb5fd65f 100644 --- a/spring-aop/src/main/java/org/springframework/aop/aspectj/annotation/BeanFactoryAspectInstanceFactory.java +++ b/spring-aop/src/main/java/org/springframework/aop/aspectj/annotation/BeanFactoryAspectInstanceFactory.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -55,7 +55,7 @@ public class BeanFactoryAspectInstanceFactory implements MetadataAwareAspectInst * Create a BeanFactoryAspectInstanceFactory. AspectJ will be called to * introspect to create AJType metadata using the type returned for the * given bean name from the BeanFactory. - * @param beanFactory BeanFactory to obtain instance(s) from + * @param beanFactory the BeanFactory to obtain instance(s) from * @param name name of the bean */ public BeanFactoryAspectInstanceFactory(BeanFactory beanFactory, String name) { @@ -66,7 +66,7 @@ public class BeanFactoryAspectInstanceFactory implements MetadataAwareAspectInst * Create a BeanFactoryAspectInstanceFactory, providing a type that AspectJ should * introspect to create AJType metadata. Use if the BeanFactory may consider the type * to be a subclass (as when using CGLIB), and the information should relate to a superclass. - * @param beanFactory BeanFactory to obtain instance(s) from + * @param beanFactory the BeanFactory to obtain instance(s) from * @param name the name of the bean * @param type the type that should be introspected by AspectJ * ({@code null} indicates resolution through {@link BeanFactory#getType} via the bean name) diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/AbstractAdvisingBeanPostProcessor.java b/spring-aop/src/main/java/org/springframework/aop/framework/AbstractAdvisingBeanPostProcessor.java index 4faca7ce2d4..21c6e8c52f2 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/AbstractAdvisingBeanPostProcessor.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/AbstractAdvisingBeanPostProcessor.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -160,7 +160,7 @@ public abstract class AbstractAdvisingBeanPostProcessor extends ProxyProcessorSu * Subclasses may choose to implement this: for example, * to change the interfaces exposed. *

The default implementation is empty. - * @param proxyFactory ProxyFactory that is already configured with + * @param proxyFactory the ProxyFactory that is already configured with * target, advisor and interfaces and will be used to create the proxy * immediately after this method returns * @since 4.2.3 diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/AbstractSingletonProxyFactoryBean.java b/spring-aop/src/main/java/org/springframework/aop/framework/AbstractSingletonProxyFactoryBean.java index f26af490880..6e18786d58b 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/AbstractSingletonProxyFactoryBean.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/AbstractSingletonProxyFactoryBean.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -54,7 +54,7 @@ public abstract class AbstractSingletonProxyFactoryBean extends ProxyConfig @Nullable private Object[] postInterceptors; - /** Default is global AdvisorAdapterRegistry */ + /** Default is global AdvisorAdapterRegistry. */ private AdvisorAdapterRegistry advisorAdapterRegistry = GlobalAdvisorAdapterRegistry.getInstance(); @Nullable diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/Advised.java b/spring-aop/src/main/java/org/springframework/aop/framework/Advised.java index db01d75a1f6..c946034bdec 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/Advised.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/Advised.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2015 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -211,7 +211,7 @@ public interface Advised extends TargetClassAware { * or -1 if no such advice is an advice for this proxy. *

The return value of this method can be used to index into * the advisors array. - * @param advice AOP Alliance advice to search for + * @param advice the AOP Alliance advice to search for * @return index from 0 of this advice, or -1 if there's no such advice */ int indexOf(Advice advice); diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/AdvisedSupport.java b/spring-aop/src/main/java/org/springframework/aop/framework/AdvisedSupport.java index 93e22e368b5..82fb1633034 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/AdvisedSupport.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/AdvisedSupport.java @@ -61,7 +61,7 @@ import org.springframework.util.CollectionUtils; */ public class AdvisedSupport extends ProxyConfig implements Advised { - /** use serialVersionUID from Spring 2.0 for interoperability */ + /** use serialVersionUID from Spring 2.0 for interoperability. */ private static final long serialVersionUID = 2651364800145442165L; @@ -72,16 +72,16 @@ public class AdvisedSupport extends ProxyConfig implements Advised { public static final TargetSource EMPTY_TARGET_SOURCE = EmptyTargetSource.INSTANCE; - /** Package-protected to allow direct access for efficiency */ + /** Package-protected to allow direct access for efficiency. */ TargetSource targetSource = EMPTY_TARGET_SOURCE; - /** Whether the Advisors are already filtered for the specific target class */ + /** Whether the Advisors are already filtered for the specific target class. */ private boolean preFiltered = false; - /** The AdvisorChainFactory to use */ + /** The AdvisorChainFactory to use. */ AdvisorChainFactory advisorChainFactory = new DefaultAdvisorChainFactory(); - /** Cache with Method as key and advisor chain List as value */ + /** Cache with Method as key and advisor chain List as value. */ private transient Map> methodCache; /** @@ -474,7 +474,7 @@ public class AdvisedSupport extends ProxyConfig implements Advised { * for the given method, based on this configuration. * @param method the proxied method * @param targetClass the target class - * @return List of MethodInterceptors (may also include InterceptorAndDynamicMethodMatchers) + * @return a List of MethodInterceptors (may also include InterceptorAndDynamicMethodMatchers) */ public List getInterceptorsAndDynamicInterceptionAdvice(Method method, @Nullable Class targetClass) { MethodCacheKey cacheKey = new MethodCacheKey(method); @@ -528,7 +528,7 @@ public class AdvisedSupport extends ProxyConfig implements Advised { /** * Build a configuration-only copy of this AdvisedSupport, - * replacing the TargetSource + * replacing the TargetSource. */ AdvisedSupport getConfigurationOnlyCopy() { AdvisedSupport copy = new AdvisedSupport(); diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/AdvisorChainFactory.java b/spring-aop/src/main/java/org/springframework/aop/framework/AdvisorChainFactory.java index 87c0a810eeb..2640101b04b 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/AdvisorChainFactory.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/AdvisorChainFactory.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2014 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -36,7 +36,7 @@ public interface AdvisorChainFactory { * @param method the proxied method * @param targetClass the target class (may be {@code null} to indicate a proxy without * target object, in which case the method's declaring class is the next best option) - * @return List of MethodInterceptors (may also include InterceptorAndDynamicMethodMatchers) + * @return a List of MethodInterceptors (may also include InterceptorAndDynamicMethodMatchers) */ List getInterceptorsAndDynamicInterceptionAdvice(Advised config, Method method, @Nullable Class targetClass); diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/AopContext.java b/spring-aop/src/main/java/org/springframework/aop/framework/AopContext.java index d37f7a958d1..796d30b8799 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/AopContext.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/AopContext.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2016 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -54,7 +54,7 @@ public abstract class AopContext { * Try to return the current AOP proxy. This method is usable only if the * calling method has been invoked via AOP, and the AOP framework has been set * to expose proxies. Otherwise, this method will throw an IllegalStateException. - * @return Object the current AOP proxy (never returns {@code null}) + * @return the current AOP proxy (never returns {@code null}) * @throws IllegalStateException if the proxy cannot be found, because the * method was invoked outside an AOP invocation context, or because the * AOP framework has not been configured to expose the proxy diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/CglibAopProxy.java b/spring-aop/src/main/java/org/springframework/aop/framework/CglibAopProxy.java index 71b2bf1ddaa..9a9b0e285ad 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/CglibAopProxy.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/CglibAopProxy.java @@ -93,14 +93,14 @@ class CglibAopProxy implements AopProxy, Serializable { private static final int INVOKE_HASHCODE = 6; - /** Logger available to subclasses; static to optimize serialization */ + /** Logger available to subclasses; static to optimize serialization. */ protected static final Log logger = LogFactory.getLog(CglibAopProxy.class); - /** Keeps track of the Classes that we have validated for final methods */ + /** Keeps track of the Classes that we have validated for final methods. */ private static final Map, Boolean> validatedClasses = new WeakHashMap<>(); - /** The configuration used to configure this proxy */ + /** The configuration used to configure this proxy. */ protected final AdvisedSupport advised; @Nullable @@ -109,7 +109,7 @@ class CglibAopProxy implements AopProxy, Serializable { @Nullable protected Class[] constructorArgTypes; - /** Dispatcher used for methods on Advised */ + /** Dispatcher used for methods on Advised. */ private final transient AdvisedDispatcher advisedDispatcher; private transient Map fixedInterceptorMap = Collections.emptyMap(); diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/JdkDynamicAopProxy.java b/spring-aop/src/main/java/org/springframework/aop/framework/JdkDynamicAopProxy.java index eb02093c204..25ed051fbe8 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/JdkDynamicAopProxy.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/JdkDynamicAopProxy.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -63,7 +63,7 @@ import org.springframework.util.ClassUtils; */ final class JdkDynamicAopProxy implements AopProxy, InvocationHandler, Serializable { - /** use serialVersionUID from Spring 1.2 for interoperability */ + /** use serialVersionUID from Spring 1.2 for interoperability. */ private static final long serialVersionUID = 5531744639992436476L; @@ -76,10 +76,10 @@ final class JdkDynamicAopProxy implements AopProxy, InvocationHandler, Serializa * This way, we can also more easily take advantage of minor optimizations in each class. */ - /** We use a static Log to avoid serialization issues */ + /** We use a static Log to avoid serialization issues. */ private static final Log logger = LogFactory.getLog(JdkDynamicAopProxy.class); - /** Config used to configure this proxy */ + /** Config used to configure this proxy. */ private final AdvisedSupport advised; /** diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/ProxyConfig.java b/spring-aop/src/main/java/org/springframework/aop/framework/ProxyConfig.java index fc7ae7b878a..9847b0de4ed 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/ProxyConfig.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/ProxyConfig.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2012 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -30,7 +30,7 @@ import org.springframework.util.Assert; */ public class ProxyConfig implements Serializable { - /** use serialVersionUID from Spring 1.2 for interoperability */ + /** use serialVersionUID from Spring 1.2 for interoperability. */ private static final long serialVersionUID = -8409359707199703185L; diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/ProxyCreatorSupport.java b/spring-aop/src/main/java/org/springframework/aop/framework/ProxyCreatorSupport.java index a0852f23dfa..ad78b04383a 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/ProxyCreatorSupport.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/ProxyCreatorSupport.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2016 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -36,7 +36,7 @@ public class ProxyCreatorSupport extends AdvisedSupport { private List listeners = new LinkedList<>(); - /** Set to true when the first AOP proxy has been created */ + /** Set to true when the first AOP proxy has been created. */ private boolean active = false; diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/ProxyFactoryBean.java b/spring-aop/src/main/java/org/springframework/aop/framework/ProxyFactoryBean.java index a806b4037e8..0c7a63b2496 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/ProxyFactoryBean.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/ProxyFactoryBean.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -124,10 +124,10 @@ public class ProxyFactoryBean extends ProxyCreatorSupport @Nullable private transient BeanFactory beanFactory; - /** Whether the advisor chain has already been initialized */ + /** Whether the advisor chain has already been initialized. */ private boolean advisorChainInitialized = false; - /** If this is a singleton, the cached singleton proxy instance */ + /** If this is a singleton, the cached singleton proxy instance. */ @Nullable private Object singletonInstance; diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/adapter/AdvisorAdapterRegistry.java b/spring-aop/src/main/java/org/springframework/aop/framework/adapter/AdvisorAdapterRegistry.java index 9eacf479bad..f15353e5b06 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/adapter/AdvisorAdapterRegistry.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/adapter/AdvisorAdapterRegistry.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2012 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -50,7 +50,7 @@ public interface AdvisorAdapterRegistry { * given Advisor in an interception-based framework. *

Don't worry about the pointcut associated with the Advisor, * if it's a PointcutAdvisor: just return an interceptor. - * @param advisor Advisor to find an interceptor for + * @param advisor the Advisor to find an interceptor for * @return an array of MethodInterceptors to expose this Advisor's behavior * @throws UnknownAdviceTypeException if the Advisor type is * not understood by any registered AdvisorAdapter. @@ -61,7 +61,7 @@ public interface AdvisorAdapterRegistry { * Register the given AdvisorAdapter. Note that it is not necessary to register * adapters for an AOP Alliance Interceptors or Spring Advices: these must be * automatically recognized by an AdvisorAdapterRegistry implementation. - * @param adapter AdvisorAdapter that understands a particular Advisor + * @param adapter the AdvisorAdapter that understands a particular Advisor * or Advice types */ void registerAdvisorAdapter(AdvisorAdapter adapter); diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/adapter/ThrowsAdviceInterceptor.java b/spring-aop/src/main/java/org/springframework/aop/framework/adapter/ThrowsAdviceInterceptor.java index 4bcd32647eb..5c0f7ee6c21 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/adapter/ThrowsAdviceInterceptor.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/adapter/ThrowsAdviceInterceptor.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -61,7 +61,7 @@ public class ThrowsAdviceInterceptor implements MethodInterceptor, AfterAdvice { private final Object throwsAdvice; - /** Methods on throws advice, keyed by exception class */ + /** Methods on throws advice, keyed by exception class. */ private final Map, Method> exceptionHandlerMap = new HashMap<>(); diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/AbstractAdvisorAutoProxyCreator.java b/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/AbstractAdvisorAutoProxyCreator.java index 2f59b4af88e..7e6db639550 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/AbstractAdvisorAutoProxyCreator.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/AbstractAdvisorAutoProxyCreator.java @@ -160,7 +160,7 @@ public abstract class AbstractAdvisorAutoProxyCreator extends AbstractAutoProxyC *

The default implementation is empty. *

Typically used to add Advisors that expose contextual information * required by some of the later advisors. - * @param candidateAdvisors Advisors that have already been identified as + * @param candidateAdvisors the Advisors that have already been identified as * applying to a given bean */ protected void extendAdvisors(List candidateAdvisors) { diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/AbstractAutoProxyCreator.java b/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/AbstractAutoProxyCreator.java index e19354723a0..4ddf79cb5c3 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/AbstractAutoProxyCreator.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/AbstractAutoProxyCreator.java @@ -109,10 +109,10 @@ public abstract class AbstractAutoProxyCreator extends ProxyProcessorSupport protected static final Object[] PROXY_WITHOUT_ADDITIONAL_INTERCEPTORS = new Object[0]; - /** Logger available to subclasses */ + /** Logger available to subclasses. */ protected final Log logger = LogFactory.getLog(getClass()); - /** Default is global AdvisorAdapterRegistry */ + /** Default is global AdvisorAdapterRegistry. */ private AdvisorAdapterRegistry advisorAdapterRegistry = GlobalAdvisorAdapterRegistry.getInstance(); /** @@ -121,7 +121,7 @@ public abstract class AbstractAutoProxyCreator extends ProxyProcessorSupport */ private boolean freezeProxy = false; - /** Default is no common interceptors */ + /** Default is no common interceptors. */ private String[] interceptorNames = new String[0]; private boolean applyCommonInterceptorsFirst = true; @@ -561,7 +561,7 @@ public abstract class AbstractAutoProxyCreator extends ProxyProcessorSupport * Subclasses may choose to implement this: for example, * to change the interfaces exposed. *

The default implementation is empty. - * @param proxyFactory ProxyFactory that is already configured with + * @param proxyFactory a ProxyFactory that is already configured with * TargetSource and interfaces and will be used to create the proxy * immediately after this method returns */ diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/DefaultAdvisorAutoProxyCreator.java b/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/DefaultAdvisorAutoProxyCreator.java index 24b8c48475e..d41a348ac61 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/DefaultAdvisorAutoProxyCreator.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/DefaultAdvisorAutoProxyCreator.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -38,7 +38,7 @@ import org.springframework.lang.Nullable; @SuppressWarnings("serial") public class DefaultAdvisorAutoProxyCreator extends AbstractAdvisorAutoProxyCreator implements BeanNameAware { - /** Separator between prefix and remainder of bean name */ + /** Separator between prefix and remainder of bean name. */ public static final String SEPARATOR = "."; diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/ProxyCreationContext.java b/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/ProxyCreationContext.java index cc7f6c3275b..0807cf47998 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/ProxyCreationContext.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/ProxyCreationContext.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2016 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -29,7 +29,7 @@ import org.springframework.lang.Nullable; */ public class ProxyCreationContext { - /** ThreadLocal holding the current proxied bean name during Advisor matching */ + /** ThreadLocal holding the current proxied bean name during Advisor matching. */ private static final ThreadLocal currentProxiedBeanName = new NamedThreadLocal<>("Name of currently proxied bean"); diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/target/AbstractBeanFactoryBasedTargetSourceCreator.java b/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/target/AbstractBeanFactoryBasedTargetSourceCreator.java index 03ec6dd1271..d161fa2ddf6 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/target/AbstractBeanFactoryBasedTargetSourceCreator.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/target/AbstractBeanFactoryBasedTargetSourceCreator.java @@ -60,7 +60,7 @@ public abstract class AbstractBeanFactoryBasedTargetSourceCreator private ConfigurableBeanFactory beanFactory; - /** Internally used DefaultListableBeanFactory instances, keyed by bean name */ + /** Internally used DefaultListableBeanFactory instances, keyed by bean name. */ private final Map internalBeanFactories = new HashMap<>(); diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/target/QuickTargetSourceCreator.java b/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/target/QuickTargetSourceCreator.java index 6101658fa45..8ddcba030d9 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/target/QuickTargetSourceCreator.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/target/QuickTargetSourceCreator.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2015 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -37,8 +37,19 @@ import org.springframework.lang.Nullable; */ public class QuickTargetSourceCreator extends AbstractBeanFactoryBasedTargetSourceCreator { + /** + * The CommonsPool2TargetSource prefix. + */ public static final String PREFIX_COMMONS_POOL = ":"; + + /** + * The ThreadLocalTargetSource prefix. + */ public static final String PREFIX_THREAD_LOCAL = "%"; + + /** + * The PrototypeTargetSource prefix. + */ public static final String PREFIX_PROTOTYPE = "!"; @Override diff --git a/spring-aop/src/main/java/org/springframework/aop/interceptor/ExposeBeanNameAdvisors.java b/spring-aop/src/main/java/org/springframework/aop/interceptor/ExposeBeanNameAdvisors.java index 78479765ca6..e12a20d0ff6 100644 --- a/spring-aop/src/main/java/org/springframework/aop/interceptor/ExposeBeanNameAdvisors.java +++ b/spring-aop/src/main/java/org/springframework/aop/interceptor/ExposeBeanNameAdvisors.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2012 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -62,7 +62,7 @@ public abstract class ExposeBeanNameAdvisors { /** * Find the bean name for the given invocation. Assumes that an ExposeBeanNameAdvisor * has been included in the interceptor chain. - * @param mi MethodInvocation that should contain the bean name as an attribute + * @param mi the MethodInvocation that should contain the bean name as an attribute * @return the bean name (never {@code null}) * @throws IllegalStateException if the bean name has not been exposed */ @@ -80,7 +80,7 @@ public abstract class ExposeBeanNameAdvisors { /** * Create a new advisor that will expose the given bean name, - * with no introduction + * with no introduction. * @param beanName bean name to expose */ public static Advisor createAdvisorWithoutIntroduction(String beanName) { diff --git a/spring-aop/src/main/java/org/springframework/aop/interceptor/ExposeInvocationInterceptor.java b/spring-aop/src/main/java/org/springframework/aop/interceptor/ExposeInvocationInterceptor.java index d4368f0ee87..581f2df8921 100644 --- a/spring-aop/src/main/java/org/springframework/aop/interceptor/ExposeInvocationInterceptor.java +++ b/spring-aop/src/main/java/org/springframework/aop/interceptor/ExposeInvocationInterceptor.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2016 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -43,7 +43,7 @@ import org.springframework.core.PriorityOrdered; @SuppressWarnings("serial") public class ExposeInvocationInterceptor implements MethodInterceptor, PriorityOrdered, Serializable { - /** Singleton instance of this class */ + /** Singleton instance of this class. */ public static final ExposeInvocationInterceptor INSTANCE = new ExposeInvocationInterceptor(); /** diff --git a/spring-aop/src/main/java/org/springframework/aop/scope/ScopedProxyFactoryBean.java b/spring-aop/src/main/java/org/springframework/aop/scope/ScopedProxyFactoryBean.java index 8889e117a79..19c35e9774e 100644 --- a/spring-aop/src/main/java/org/springframework/aop/scope/ScopedProxyFactoryBean.java +++ b/spring-aop/src/main/java/org/springframework/aop/scope/ScopedProxyFactoryBean.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -54,14 +54,14 @@ import org.springframework.util.ClassUtils; @SuppressWarnings("serial") public class ScopedProxyFactoryBean extends ProxyConfig implements FactoryBean, BeanFactoryAware { - /** The TargetSource that manages scoping */ + /** The TargetSource that manages scoping. */ private final SimpleBeanTargetSource scopedTargetSource = new SimpleBeanTargetSource(); - /** The name of the target bean */ + /** The name of the target bean. */ @Nullable private String targetBeanName; - /** The cached singleton proxy */ + /** The cached singleton proxy. */ @Nullable private Object proxy; diff --git a/spring-aop/src/main/java/org/springframework/aop/support/ComposablePointcut.java b/spring-aop/src/main/java/org/springframework/aop/support/ComposablePointcut.java index 50a67d2ae2f..501d8194b65 100644 --- a/spring-aop/src/main/java/org/springframework/aop/support/ComposablePointcut.java +++ b/spring-aop/src/main/java/org/springframework/aop/support/ComposablePointcut.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -39,7 +39,7 @@ import org.springframework.util.Assert; */ public class ComposablePointcut implements Pointcut, Serializable { - /** use serialVersionUID from Spring 1.2 for interoperability */ + /** use serialVersionUID from Spring 1.2 for interoperability. */ private static final long serialVersionUID = -2743223737633663832L; private ClassFilter classFilter; diff --git a/spring-aop/src/main/java/org/springframework/aop/support/DynamicMethodMatcher.java b/spring-aop/src/main/java/org/springframework/aop/support/DynamicMethodMatcher.java index b46b0e4912f..8ae7e821102 100644 --- a/spring-aop/src/main/java/org/springframework/aop/support/DynamicMethodMatcher.java +++ b/spring-aop/src/main/java/org/springframework/aop/support/DynamicMethodMatcher.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2012 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -24,6 +24,8 @@ import org.springframework.lang.Nullable; /** * Convenient abstract superclass for dynamic method matchers, * which do care about arguments at runtime. + * + * @author Rod Johnson */ public abstract class DynamicMethodMatcher implements MethodMatcher { diff --git a/spring-aop/src/main/java/org/springframework/aop/support/Pointcuts.java b/spring-aop/src/main/java/org/springframework/aop/support/Pointcuts.java index 42475d9bb82..8af45e2e145 100644 --- a/spring-aop/src/main/java/org/springframework/aop/support/Pointcuts.java +++ b/spring-aop/src/main/java/org/springframework/aop/support/Pointcuts.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2015 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -35,10 +35,10 @@ import org.springframework.util.Assert; */ public abstract class Pointcuts { - /** Pointcut matching all bean property setters, in any class */ + /** Pointcut matching all bean property setters, in any class. */ public static final Pointcut SETTERS = SetterPointcut.INSTANCE; - /** Pointcut matching all bean property getters, in any class */ + /** Pointcut matching all bean property getters, in any class. */ public static final Pointcut GETTERS = GetterPointcut.INSTANCE; diff --git a/spring-aop/src/main/java/org/springframework/aop/support/RootClassFilter.java b/spring-aop/src/main/java/org/springframework/aop/support/RootClassFilter.java index 364ff6282ec..7c4ba255faa 100644 --- a/spring-aop/src/main/java/org/springframework/aop/support/RootClassFilter.java +++ b/spring-aop/src/main/java/org/springframework/aop/support/RootClassFilter.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2012 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -21,7 +21,8 @@ import java.io.Serializable; import org.springframework.aop.ClassFilter; /** - * Simple ClassFilter implementation that passes classes (and optionally subclasses) + * Simple ClassFilter implementation that passes classes (and optionally subclasses). + * * @author Rod Johnson */ @SuppressWarnings("serial") diff --git a/spring-aop/src/main/java/org/springframework/aop/support/StaticMethodMatcher.java b/spring-aop/src/main/java/org/springframework/aop/support/StaticMethodMatcher.java index 76f89ab3eb4..150ac87c98f 100644 --- a/spring-aop/src/main/java/org/springframework/aop/support/StaticMethodMatcher.java +++ b/spring-aop/src/main/java/org/springframework/aop/support/StaticMethodMatcher.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2015 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -24,6 +24,8 @@ import org.springframework.lang.Nullable; /** * Convenient abstract superclass for static method matchers, which don't care * about arguments at runtime. + * + * @author Rod Johnson */ public abstract class StaticMethodMatcher implements MethodMatcher { diff --git a/spring-aop/src/main/java/org/springframework/aop/target/AbstractBeanFactoryBasedTargetSource.java b/spring-aop/src/main/java/org/springframework/aop/target/AbstractBeanFactoryBasedTargetSource.java index b070b628ead..55422227406 100644 --- a/spring-aop/src/main/java/org/springframework/aop/target/AbstractBeanFactoryBasedTargetSource.java +++ b/spring-aop/src/main/java/org/springframework/aop/target/AbstractBeanFactoryBasedTargetSource.java @@ -49,17 +49,17 @@ import org.springframework.util.ObjectUtils; */ public abstract class AbstractBeanFactoryBasedTargetSource implements TargetSource, BeanFactoryAware, Serializable { - /** use serialVersionUID from Spring 1.2.7 for interoperability */ + /** use serialVersionUID from Spring 1.2.7 for interoperability. */ private static final long serialVersionUID = -4721607536018568393L; - /** Logger available to subclasses */ + /** Logger available to subclasses. */ protected final Log logger = LogFactory.getLog(getClass()); - /** Name of the target bean we will create on each invocation */ + /** Name of the target bean we will create on each invocation. */ private String targetBeanName; - /** Class of the target */ + /** Class of the target. */ private volatile Class targetClass; /** diff --git a/spring-aop/src/main/java/org/springframework/aop/target/AbstractLazyCreationTargetSource.java b/spring-aop/src/main/java/org/springframework/aop/target/AbstractLazyCreationTargetSource.java index 4c23c54c365..81dbad3e3d4 100644 --- a/spring-aop/src/main/java/org/springframework/aop/target/AbstractLazyCreationTargetSource.java +++ b/spring-aop/src/main/java/org/springframework/aop/target/AbstractLazyCreationTargetSource.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2012 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -42,10 +42,10 @@ import org.springframework.lang.Nullable; */ public abstract class AbstractLazyCreationTargetSource implements TargetSource { - /** Logger available to subclasses */ + /** Logger available to subclasses. */ protected final Log logger = LogFactory.getLog(getClass()); - /** The lazily initialized target object */ + /** The lazily initialized target object. */ private Object lazyTarget; diff --git a/spring-aop/src/main/java/org/springframework/aop/target/AbstractPoolingTargetSource.java b/spring-aop/src/main/java/org/springframework/aop/target/AbstractPoolingTargetSource.java index bfd4a5f147d..ccdb5c7bb26 100644 --- a/spring-aop/src/main/java/org/springframework/aop/target/AbstractPoolingTargetSource.java +++ b/spring-aop/src/main/java/org/springframework/aop/target/AbstractPoolingTargetSource.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2015 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -55,7 +55,7 @@ import org.springframework.lang.Nullable; public abstract class AbstractPoolingTargetSource extends AbstractPrototypeBasedTargetSource implements PoolingConfig, DisposableBean { - /** The maximum size of the pool */ + /** The maximum size of the pool. */ private int maxSize = -1; diff --git a/spring-aop/src/main/java/org/springframework/aop/target/CommonsPool2TargetSource.java b/spring-aop/src/main/java/org/springframework/aop/target/CommonsPool2TargetSource.java index d0f891c8b85..8a6fcb8aae2 100644 --- a/spring-aop/src/main/java/org/springframework/aop/target/CommonsPool2TargetSource.java +++ b/spring-aop/src/main/java/org/springframework/aop/target/CommonsPool2TargetSource.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -79,7 +79,7 @@ public class CommonsPool2TargetSource extends AbstractPoolingTargetSource implem private boolean blockWhenExhausted = GenericObjectPoolConfig.DEFAULT_BLOCK_WHEN_EXHAUSTED; /** - * The Apache Commons {@code ObjectPool} used to pool target objects + * The Apache Commons {@code ObjectPool} used to pool target objects. */ @Nullable private ObjectPool pool; diff --git a/spring-aop/src/main/java/org/springframework/aop/target/EmptyTargetSource.java b/spring-aop/src/main/java/org/springframework/aop/target/EmptyTargetSource.java index 344d33a8bb3..4dd24146a3e 100644 --- a/spring-aop/src/main/java/org/springframework/aop/target/EmptyTargetSource.java +++ b/spring-aop/src/main/java/org/springframework/aop/target/EmptyTargetSource.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -32,7 +32,7 @@ import org.springframework.util.ObjectUtils; */ public class EmptyTargetSource implements TargetSource, Serializable { - /** use serialVersionUID from Spring 1.2 for interoperability */ + /** use serialVersionUID from Spring 1.2 for interoperability. */ private static final long serialVersionUID = 3680494563553489691L; diff --git a/spring-aop/src/main/java/org/springframework/aop/target/HotSwappableTargetSource.java b/spring-aop/src/main/java/org/springframework/aop/target/HotSwappableTargetSource.java index 9f7e5fc2639..f56c71d2c35 100644 --- a/spring-aop/src/main/java/org/springframework/aop/target/HotSwappableTargetSource.java +++ b/spring-aop/src/main/java/org/springframework/aop/target/HotSwappableTargetSource.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2012 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -37,11 +37,11 @@ import org.springframework.util.Assert; */ public class HotSwappableTargetSource implements TargetSource, Serializable { - /** use serialVersionUID from Spring 1.2 for interoperability */ + /** use serialVersionUID from Spring 1.2 for interoperability. */ private static final long serialVersionUID = 7497929212653839187L; - /** The current target object */ + /** The current target object. */ private Object target; diff --git a/spring-aop/src/main/java/org/springframework/aop/target/SingletonTargetSource.java b/spring-aop/src/main/java/org/springframework/aop/target/SingletonTargetSource.java index 439e59509a8..6544ab3daff 100644 --- a/spring-aop/src/main/java/org/springframework/aop/target/SingletonTargetSource.java +++ b/spring-aop/src/main/java/org/springframework/aop/target/SingletonTargetSource.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2012 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -37,11 +37,11 @@ import org.springframework.util.ObjectUtils; */ public class SingletonTargetSource implements TargetSource, Serializable { - /** use serialVersionUID from Spring 1.2 for interoperability */ + /** use serialVersionUID from Spring 1.2 for interoperability. */ private static final long serialVersionUID = 9031246629662423738L; - /** Target cached and invoked using reflection */ + /** Target cached and invoked using reflection. */ private final Object target; diff --git a/spring-aop/src/main/java/org/springframework/aop/target/dynamic/AbstractRefreshableTargetSource.java b/spring-aop/src/main/java/org/springframework/aop/target/dynamic/AbstractRefreshableTargetSource.java index 75d698db9d8..8cba76b0f7f 100644 --- a/spring-aop/src/main/java/org/springframework/aop/target/dynamic/AbstractRefreshableTargetSource.java +++ b/spring-aop/src/main/java/org/springframework/aop/target/dynamic/AbstractRefreshableTargetSource.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2016 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -39,7 +39,7 @@ import org.springframework.lang.Nullable; */ public abstract class AbstractRefreshableTargetSource implements TargetSource, Refreshable { - /** Logger available to subclasses */ + /** Logger available to subclasses. */ protected final Log logger = LogFactory.getLog(getClass()); @Nullable diff --git a/spring-aspects/src/main/java/org/springframework/context/annotation/aspectj/SpringConfiguredConfiguration.java b/spring-aspects/src/main/java/org/springframework/context/annotation/aspectj/SpringConfiguredConfiguration.java index 1d141b2ee92..05cd5e4caf3 100644 --- a/spring-aspects/src/main/java/org/springframework/context/annotation/aspectj/SpringConfiguredConfiguration.java +++ b/spring-aspects/src/main/java/org/springframework/context/annotation/aspectj/SpringConfiguredConfiguration.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2015 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -39,6 +39,9 @@ import org.springframework.context.annotation.Role; @Configuration public class SpringConfiguredConfiguration { + /** + * The bean name used for the configurer aspect. + */ public static final String BEAN_CONFIGURER_ASPECT_BEAN_NAME = "org.springframework.context.config.internalBeanConfigurerAspect"; diff --git a/spring-beans/src/main/java/org/springframework/beans/AbstractNestablePropertyAccessor.java b/spring-beans/src/main/java/org/springframework/beans/AbstractNestablePropertyAccessor.java index 5b6ef07aac1..68fe04db98e 100644 --- a/spring-beans/src/main/java/org/springframework/beans/AbstractNestablePropertyAccessor.java +++ b/spring-beans/src/main/java/org/springframework/beans/AbstractNestablePropertyAccessor.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -86,7 +86,7 @@ public abstract class AbstractNestablePropertyAccessor extends AbstractPropertyA @Nullable Object rootObject; - /** Map with cached nested Accessors: nested path -> Accessor instance */ + /** Map with cached nested Accessors: nested path -> Accessor instance. */ @Nullable private Map nestedPropertyAccessors; @@ -971,6 +971,9 @@ public abstract class AbstractNestablePropertyAccessor extends AbstractPropertyA } + /** + * A handler for a specific property. + */ protected abstract static class PropertyHandler { private final Class propertyType; @@ -1026,6 +1029,9 @@ public abstract class AbstractNestablePropertyAccessor extends AbstractPropertyA } + /** + * Holder class used to store property tokens. + */ protected static class PropertyTokenHolder { public PropertyTokenHolder(String name) { diff --git a/spring-beans/src/main/java/org/springframework/beans/BeanUtils.java b/spring-beans/src/main/java/org/springframework/beans/BeanUtils.java index f9712f4af27..7c781585162 100644 --- a/spring-beans/src/main/java/org/springframework/beans/BeanUtils.java +++ b/spring-beans/src/main/java/org/springframework/beans/BeanUtils.java @@ -401,7 +401,8 @@ public abstract class BeanUtils { /** - * Retrieve the JavaBeans {@code PropertyDescriptor}s of a given class. + * Retrieve the JavaBeans {@code PropertyDescriptor}s of a given + * class. * @param clazz the Class to retrieve the PropertyDescriptors for * @return an array of {@code PropertyDescriptors} for the given class * @throws BeansException if PropertyDescriptor look fails diff --git a/spring-beans/src/main/java/org/springframework/beans/BeanWrapperImpl.java b/spring-beans/src/main/java/org/springframework/beans/BeanWrapperImpl.java index 7aaf0d470db..eb8469a86d6 100644 --- a/spring-beans/src/main/java/org/springframework/beans/BeanWrapperImpl.java +++ b/spring-beans/src/main/java/org/springframework/beans/BeanWrapperImpl.java @@ -70,7 +70,7 @@ public class BeanWrapperImpl extends AbstractNestablePropertyAccessor implements private CachedIntrospectionResults cachedIntrospectionResults; /** - * The security context used for invoking the property methods + * The security context used for invoking the property methods. */ @Nullable private AccessControlContext acc; diff --git a/spring-beans/src/main/java/org/springframework/beans/CachedIntrospectionResults.java b/spring-beans/src/main/java/org/springframework/beans/CachedIntrospectionResults.java index 8656d9e7023..2659293560f 100644 --- a/spring-beans/src/main/java/org/springframework/beans/CachedIntrospectionResults.java +++ b/spring-beans/src/main/java/org/springframework/beans/CachedIntrospectionResults.java @@ -96,7 +96,7 @@ public class CachedIntrospectionResults { private static final boolean shouldIntrospectorIgnoreBeaninfoClasses = SpringProperties.getFlag(IGNORE_BEANINFO_PROPERTY_NAME); - /** Stores the BeanInfoFactory instances */ + /** Stores the BeanInfoFactory instances. */ private static List beanInfoFactories = SpringFactoriesLoader.loadFactories( BeanInfoFactory.class, CachedIntrospectionResults.class.getClassLoader()); @@ -250,13 +250,13 @@ public class CachedIntrospectionResults { } - /** The BeanInfo object for the introspected bean class */ + /** The BeanInfo object for the introspected bean class. */ private final BeanInfo beanInfo; - /** PropertyDescriptor objects keyed by property name String */ + /** PropertyDescriptor objects keyed by property name String. */ private final Map propertyDescriptorCache; - /** TypeDescriptor objects keyed by PropertyDescriptor */ + /** TypeDescriptor objects keyed by PropertyDescriptor. */ private final ConcurrentMap typeDescriptorCache; diff --git a/spring-beans/src/main/java/org/springframework/beans/ExtendedBeanInfo.java b/spring-beans/src/main/java/org/springframework/beans/ExtendedBeanInfo.java index 2f87b10a365..4571190b2b2 100644 --- a/spring-beans/src/main/java/org/springframework/beans/ExtendedBeanInfo.java +++ b/spring-beans/src/main/java/org/springframework/beans/ExtendedBeanInfo.java @@ -213,9 +213,9 @@ class ExtendedBeanInfo implements BeanInfo { /** - * Return the set of {@link PropertyDescriptor}s from the wrapped {@link BeanInfo} - * object as well as {@code PropertyDescriptor}s for each non-void returning setter - * method found during construction. + * Return the set of {@link PropertyDescriptor PropertyDescriptors} from the wrapped + * {@link BeanInfo} object as well as {@code PropertyDescriptor BeanInfo} object as well as {@code PropertyDescriptors} + * for each non-void returning setter method found during construction. * @see #ExtendedBeanInfo(BeanInfo) */ @Override @@ -259,6 +259,9 @@ class ExtendedBeanInfo implements BeanInfo { } + /** + * A simple {@link PropertyDescriptor}. + */ static class SimplePropertyDescriptor extends PropertyDescriptor { @Nullable @@ -350,6 +353,9 @@ class ExtendedBeanInfo implements BeanInfo { } + /** + * A simple {@link IndexedPropertyDescriptor}. + */ static class SimpleIndexedPropertyDescriptor extends IndexedPropertyDescriptor { @Nullable diff --git a/spring-beans/src/main/java/org/springframework/beans/MethodInvocationException.java b/spring-beans/src/main/java/org/springframework/beans/MethodInvocationException.java index e82daabbcb1..c6e3e411ab8 100644 --- a/spring-beans/src/main/java/org/springframework/beans/MethodInvocationException.java +++ b/spring-beans/src/main/java/org/springframework/beans/MethodInvocationException.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2012 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -35,7 +35,7 @@ public class MethodInvocationException extends PropertyAccessException { /** * Create a new MethodInvocationException. - * @param propertyChangeEvent PropertyChangeEvent that resulted in an exception + * @param propertyChangeEvent the PropertyChangeEvent that resulted in an exception * @param cause the Throwable raised by the invoked method */ public MethodInvocationException(PropertyChangeEvent propertyChangeEvent, Throwable cause) { diff --git a/spring-beans/src/main/java/org/springframework/beans/MutablePropertyValues.java b/spring-beans/src/main/java/org/springframework/beans/MutablePropertyValues.java index 64593ad7fa3..f173bf70b48 100644 --- a/spring-beans/src/main/java/org/springframework/beans/MutablePropertyValues.java +++ b/spring-beans/src/main/java/org/springframework/beans/MutablePropertyValues.java @@ -80,7 +80,7 @@ public class MutablePropertyValues implements PropertyValues, Serializable { /** * Construct a new MutablePropertyValues object from a Map. - * @param original Map with property values keyed by property name Strings + * @param original a Map with property values keyed by property name Strings * @see #addPropertyValues(Map) */ public MutablePropertyValues(@Nullable Map original) { @@ -101,7 +101,7 @@ public class MutablePropertyValues implements PropertyValues, Serializable { * PropertyValue objects as-is. *

This is a constructor for advanced usage scenarios. * It is not intended for typical programmatic use. - * @param propertyValueList List of PropertyValue objects + * @param propertyValueList a List of PropertyValue objects */ public MutablePropertyValues(@Nullable List propertyValueList) { this.propertyValueList = @@ -145,7 +145,7 @@ public class MutablePropertyValues implements PropertyValues, Serializable { /** * Add all property values from the given Map. - * @param other Map with property values keyed by property name, + * @param other a Map with property values keyed by property name, * which must be a String * @return this in order to allow for adding multiple property values in a chain */ @@ -160,7 +160,7 @@ public class MutablePropertyValues implements PropertyValues, Serializable { /** * Add a PropertyValue object, replacing any existing one for the * corresponding property or getting merged with it (if applicable). - * @param pv PropertyValue object to add + * @param pv the PropertyValue object to add * @return this in order to allow for adding multiple property values in a chain */ public MutablePropertyValues addPropertyValue(PropertyValue pv) { diff --git a/spring-beans/src/main/java/org/springframework/beans/PropertyAccessor.java b/spring-beans/src/main/java/org/springframework/beans/PropertyAccessor.java index 960118126cd..15d3be496b7 100644 --- a/spring-beans/src/main/java/org/springframework/beans/PropertyAccessor.java +++ b/spring-beans/src/main/java/org/springframework/beans/PropertyAccessor.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -39,6 +39,11 @@ public interface PropertyAccessor { * Follows normal Java conventions: getFoo().getBar() would be "foo.bar". */ String NESTED_PROPERTY_SEPARATOR = "."; + + /** + * Path separator for nested properties. + * Follows normal Java conventions: getFoo().getBar() would be "foo.bar". + */ char NESTED_PROPERTY_SEPARATOR_CHAR = '.'; /** @@ -46,6 +51,11 @@ public interface PropertyAccessor { * indexed or mapped property like "person.addresses[0]". */ String PROPERTY_KEY_PREFIX = "["; + + /** + * Marker that indicates the start of a property key for an + * indexed or mapped property like "person.addresses[0]". + */ char PROPERTY_KEY_PREFIX_CHAR = '['; /** @@ -53,6 +63,11 @@ public interface PropertyAccessor { * indexed or mapped property like "person.addresses[0]". */ String PROPERTY_KEY_SUFFIX = "]"; + + /** + * Marker that indicates the end of a property key for an + * indexed or mapped property like "person.addresses[0]". + */ char PROPERTY_KEY_SUFFIX_CHAR = ']'; @@ -141,7 +156,7 @@ public interface PropertyAccessor { *

Bulk updates from PropertyValues are more powerful: This method is * provided for convenience. Behavior will be identical to that of * the {@link #setPropertyValues(PropertyValues)} method. - * @param map Map to take properties from. Contains property value objects, + * @param map a Map to take properties from. Contains property value objects, * keyed by property name * @throws InvalidPropertyException if there is no such property or * if the property isn't writable @@ -162,7 +177,7 @@ public interface PropertyAccessor { * This exception can be examined later to see all binding errors. * Properties that were successfully updated remain changed. *

Does not allow unknown fields or invalid fields. - * @param pvs PropertyValues to set on the target object + * @param pvs a PropertyValues to set on the target object * @throws InvalidPropertyException if there is no such property or * if the property isn't writable * @throws PropertyBatchUpdateException if one or more PropertyAccessExceptions @@ -182,7 +197,7 @@ public interface PropertyAccessor { * {@link PropertyBatchUpdateException} containing all the individual errors. * This exception can be examined later to see all binding errors. * Properties that were successfully updated remain changed. - * @param pvs PropertyValues to set on the target object + * @param pvs a PropertyValues to set on the target object * @param ignoreUnknown should we ignore unknown properties (not found in the bean) * @throws InvalidPropertyException if there is no such property or * if the property isn't writable @@ -204,7 +219,7 @@ public interface PropertyAccessor { * {@link PropertyBatchUpdateException} containing all the individual errors. * This exception can be examined later to see all binding errors. * Properties that were successfully updated remain changed. - * @param pvs PropertyValues to set on the target object + * @param pvs a PropertyValues to set on the target object * @param ignoreUnknown should we ignore unknown properties (not found in the bean) * @param ignoreInvalid should we ignore invalid properties (found but not accessible) * @throws InvalidPropertyException if there is no such property or diff --git a/spring-beans/src/main/java/org/springframework/beans/PropertyBatchUpdateException.java b/spring-beans/src/main/java/org/springframework/beans/PropertyBatchUpdateException.java index b3bdfeabdd3..943deabe33f 100644 --- a/spring-beans/src/main/java/org/springframework/beans/PropertyBatchUpdateException.java +++ b/spring-beans/src/main/java/org/springframework/beans/PropertyBatchUpdateException.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -39,7 +39,7 @@ import org.springframework.util.ObjectUtils; @SuppressWarnings("serial") public class PropertyBatchUpdateException extends BeansException { - /** List of PropertyAccessException objects */ + /** List of PropertyAccessException objects. */ private PropertyAccessException[] propertyAccessExceptions; diff --git a/spring-beans/src/main/java/org/springframework/beans/PropertyMatches.java b/spring-beans/src/main/java/org/springframework/beans/PropertyMatches.java index f37275ba18c..1c41ea9ee37 100644 --- a/spring-beans/src/main/java/org/springframework/beans/PropertyMatches.java +++ b/spring-beans/src/main/java/org/springframework/beans/PropertyMatches.java @@ -42,7 +42,7 @@ import org.springframework.util.StringUtils; */ public abstract class PropertyMatches { - /** Default maximum property distance: 2 */ + /** Default maximum property distance: 2. */ public static final int DEFAULT_MAX_DISTANCE = 2; diff --git a/spring-beans/src/main/java/org/springframework/beans/PropertyValue.java b/spring-beans/src/main/java/org/springframework/beans/PropertyValue.java index 1f1536fc1b9..11086f54254 100644 --- a/spring-beans/src/main/java/org/springframework/beans/PropertyValue.java +++ b/spring-beans/src/main/java/org/springframework/beans/PropertyValue.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -54,11 +54,11 @@ public class PropertyValue extends BeanMetadataAttributeAccessor implements Seri @Nullable private Object convertedValue; - /** Package-visible field that indicates whether conversion is necessary */ + /** Package-visible field that indicates whether conversion is necessary. */ @Nullable volatile Boolean conversionNecessary; - /** Package-visible field for caching the resolved property path tokens */ + /** Package-visible field for caching the resolved property path tokens. */ @Nullable transient volatile Object resolvedTokens; diff --git a/spring-beans/src/main/java/org/springframework/beans/PropertyValues.java b/spring-beans/src/main/java/org/springframework/beans/PropertyValues.java index 786354aeb83..8e0abb14276 100644 --- a/spring-beans/src/main/java/org/springframework/beans/PropertyValues.java +++ b/spring-beans/src/main/java/org/springframework/beans/PropertyValues.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -46,7 +46,7 @@ public interface PropertyValues { * Return the changes since the previous PropertyValues. * Subclasses should also override {@code equals}. * @param old old property values - * @return PropertyValues updated or new properties. + * @return the updated or new properties. * Return empty PropertyValues if there are no changes. * @see Object#equals */ diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/BeanNotOfRequiredTypeException.java b/spring-beans/src/main/java/org/springframework/beans/factory/BeanNotOfRequiredTypeException.java index 1cb0b75830e..5e552b861e5 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/BeanNotOfRequiredTypeException.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/BeanNotOfRequiredTypeException.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2016 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -28,13 +28,13 @@ import org.springframework.util.ClassUtils; @SuppressWarnings("serial") public class BeanNotOfRequiredTypeException extends BeansException { - /** The name of the instance that was of the wrong type */ + /** The name of the instance that was of the wrong type. */ private String beanName; - /** The required type */ + /** The required type. */ private Class requiredType; - /** The offending type */ + /** The offending type. */ private Class actualType; diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/FactoryBean.java b/spring-beans/src/main/java/org/springframework/beans/factory/FactoryBean.java index 78c008c2977..4746ada3818 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/FactoryBean.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/FactoryBean.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2016 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -51,6 +51,7 @@ import org.springframework.lang.Nullable; * @author Rod Johnson * @author Juergen Hoeller * @since 08.03.2003 + * @param the bean type * @see org.springframework.beans.factory.BeanFactory * @see org.springframework.aop.framework.ProxyFactoryBean * @see org.springframework.jndi.JndiObjectFactoryBean diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/ObjectFactory.java b/spring-beans/src/main/java/org/springframework/beans/factory/ObjectFactory.java index 9546b6ee736..06ee95c0a3d 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/ObjectFactory.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/ObjectFactory.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -33,6 +33,7 @@ import org.springframework.beans.BeansException; * * @author Colin Sampaleanu * @since 1.0.2 + * @param the object type * @see FactoryBean */ @FunctionalInterface diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/ObjectProvider.java b/spring-beans/src/main/java/org/springframework/beans/factory/ObjectProvider.java index 527aa30816b..209444bcfa2 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/ObjectProvider.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/ObjectProvider.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -28,6 +28,7 @@ import org.springframework.lang.Nullable; * * @author Juergen Hoeller * @since 4.3 + * @param the object type */ public interface ObjectProvider extends ObjectFactory { diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/SmartFactoryBean.java b/spring-beans/src/main/java/org/springframework/beans/factory/SmartFactoryBean.java index 1f16eeb00b3..42cc6e48706 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/SmartFactoryBean.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/SmartFactoryBean.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2016 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -35,6 +35,7 @@ package org.springframework.beans.factory; * * @author Juergen Hoeller * @since 2.0.3 + * @param the bean type * @see #isPrototype() * @see #isSingleton() */ diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/annotation/InjectionMetadata.java b/spring-beans/src/main/java/org/springframework/beans/factory/annotation/InjectionMetadata.java index 58c7296f6f8..a25a5f4023a 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/annotation/InjectionMetadata.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/annotation/InjectionMetadata.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -94,6 +94,7 @@ public class InjectionMetadata { } /** + * Clear property skipping for the contained elements. * @since 3.2.13 */ public void clear(@Nullable PropertyValues pvs) { @@ -113,6 +114,9 @@ public class InjectionMetadata { } + /** + * A single injected element. + */ public abstract static class InjectedElement { protected final Member member; @@ -226,6 +230,7 @@ public class InjectionMetadata { } /** + * Clear property skipping for this element. * @since 3.2.13 */ protected void clearPropertySkipping(@Nullable PropertyValues pvs) { diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/annotation/RequiredAnnotationBeanPostProcessor.java b/spring-beans/src/main/java/org/springframework/beans/factory/annotation/RequiredAnnotationBeanPostProcessor.java index 273f77410c1..7637f5460d4 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/annotation/RequiredAnnotationBeanPostProcessor.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/annotation/RequiredAnnotationBeanPostProcessor.java @@ -93,7 +93,7 @@ public class RequiredAnnotationBeanPostProcessor extends InstantiationAwareBeanP private ConfigurableListableBeanFactory beanFactory; /** - * Cache for validated bean names, skipping re-validation for the same bean + * Cache for validated bean names, skipping re-validation for the same bean. */ private final Set validatedBeanNames = Collections.newSetFromMap(new ConcurrentHashMap<>(64)); diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/config/AbstractFactoryBean.java b/spring-beans/src/main/java/org/springframework/beans/factory/config/AbstractFactoryBean.java index d589302e9dd..483cc638450 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/config/AbstractFactoryBean.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/config/AbstractFactoryBean.java @@ -56,13 +56,14 @@ import org.springframework.util.ReflectionUtils; * @author Juergen Hoeller * @author Keith Donald * @since 1.0.2 + * @param the bean type * @see #setSingleton * @see #createInstance() */ public abstract class AbstractFactoryBean implements FactoryBean, BeanClassLoaderAware, BeanFactoryAware, InitializingBean, DisposableBean { - /** Logger available to subclasses */ + /** Logger available to subclasses. */ protected final Log logger = LogFactory.getLog(getClass()); private boolean singleton = true; diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/config/AutowireCapableBeanFactory.java b/spring-beans/src/main/java/org/springframework/beans/factory/config/AutowireCapableBeanFactory.java index 9facb43e188..96c0f6d2b93 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/config/AutowireCapableBeanFactory.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/config/AutowireCapableBeanFactory.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -118,7 +118,7 @@ public interface AutowireCapableBeanFactory extends BeanFactory { * {@link BeanPostProcessor BeanPostProcessors}. *

Note: This is intended for creating a fresh instance, populating annotated * fields and methods as well as applying all standard bean initialization callbacks. - * It does not imply traditional by-name or by-type autowiring of properties; + * It does not imply traditional by-name or by-type autowiring of properties; * use {@link #createBean(Class, int, boolean)} for those purposes. * @param beanClass the class of the bean to create * @return the new bean instance diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/config/ConfigurableBeanFactory.java b/spring-beans/src/main/java/org/springframework/beans/factory/config/ConfigurableBeanFactory.java index a4f4b6b733d..ba1c0901bdb 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/config/ConfigurableBeanFactory.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/config/ConfigurableBeanFactory.java @@ -194,9 +194,9 @@ public interface ConfigurableBeanFactory extends HierarchicalBeanFactory, Single * bean property values, constructor argument values, etc. *

This will override the default PropertyEditor mechanism and hence make * any custom editors or custom editor registrars irrelevant. + * @since 2.5 * @see #addPropertyEditorRegistrar * @see #registerCustomEditor - * @since 2.5 */ void setTypeConverter(TypeConverter typeConverter); diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/config/CustomEditorConfigurer.java b/spring-beans/src/main/java/org/springframework/beans/factory/config/CustomEditorConfigurer.java index 42b2efd696c..aeaf23fc041 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/config/CustomEditorConfigurer.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/config/CustomEditorConfigurer.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -68,10 +68,10 @@ import org.springframework.util.ClassUtils; * *

* Note, that you shouldn't register {@link PropertyEditor} bean instances via - * the {@code customEditors} property as {@link PropertyEditor}s are stateful + * the {@code customEditors} property as {@link PropertyEditor PropertyEditors} are stateful * and the instances will then have to be synchronized for every editing * attempt. In case you need control over the instantiation process of - * {@link PropertyEditor}s, use a {@link PropertyEditorRegistrar} to register + * {@link PropertyEditor PropertyEditors}, use a {@link PropertyEditorRegistrar} to register * them. * *

diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/config/FieldRetrievingFactoryBean.java b/spring-beans/src/main/java/org/springframework/beans/factory/config/FieldRetrievingFactoryBean.java index c537ee2f85b..dc81576cb2b 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/config/FieldRetrievingFactoryBean.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/config/FieldRetrievingFactoryBean.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -35,14 +35,15 @@ import org.springframework.util.StringUtils; * *

Typically used for retrieving public static final constants. Usage example: * - * 

// standard definition for exposing a static field, specifying the "staticField" property
+ * 
+ * // standard definition for exposing a static field, specifying the "staticField" property
  * <bean id="myField" class="org.springframework.beans.factory.config.FieldRetrievingFactoryBean">
  *   <property name="staticField" value="java.sql.Connection.TRANSACTION_SERIALIZABLE"/>
  * </bean>
  *
  * // convenience version that specifies a static field pattern as bean name
  * <bean id="java.sql.Connection.TRANSACTION_SERIALIZABLE"
- *       class="org.springframework.beans.factory.config.FieldRetrievingFactoryBean"/>

+ *       class="org.springframework.beans.factory.config.FieldRetrievingFactoryBean"/>
  *
* *

If you are using Spring 2.0, you can also use the following style of configuration for diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/config/MethodInvokingFactoryBean.java b/spring-beans/src/main/java/org/springframework/beans/factory/config/MethodInvokingFactoryBean.java index f3d1243f23a..8885daee6fc 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/config/MethodInvokingFactoryBean.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/config/MethodInvokingFactoryBean.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -87,7 +87,7 @@ public class MethodInvokingFactoryBean extends MethodInvokingBean implements Fac private boolean initialized = false; - /** Method call result in the singleton case */ + /** Method call result in the singleton case. */ @Nullable private Object singletonObject; diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/config/NamedBeanHolder.java b/spring-beans/src/main/java/org/springframework/beans/factory/config/NamedBeanHolder.java index 03981b12321..74ab18ffa89 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/config/NamedBeanHolder.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/config/NamedBeanHolder.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -24,6 +24,7 @@ import org.springframework.util.Assert; * * @author Juergen Hoeller * @since 4.3.3 + * @param the bean type * @see AutowireCapableBeanFactory#resolveNamedBean(Class) */ public class NamedBeanHolder implements NamedBean { diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/config/PlaceholderConfigurerSupport.java b/spring-beans/src/main/java/org/springframework/beans/factory/config/PlaceholderConfigurerSupport.java index 89da05a4a1e..c9d0f828694 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/config/PlaceholderConfigurerSupport.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/config/PlaceholderConfigurerSupport.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -36,10 +36,10 @@ import org.springframework.util.StringValueResolver; * Example XML bean definition: * * 

- * 
- *   
- *   
- * 
+ * <bean id="dataSource" class="org.springframework.jdbc.datasource.DriverManagerDataSource"/>
+ *   <property name="driverClassName" value="${driver}"/>
+ *   <property name="url" value="jdbc:${dbname}"/>
+ * </bean>
  *
* * Example properties file: @@ -89,23 +89,23 @@ import org.springframework.util.StringValueResolver; public abstract class PlaceholderConfigurerSupport extends PropertyResourceConfigurer implements BeanNameAware, BeanFactoryAware { - /** Default placeholder prefix: {@value} */ + /** Default placeholder prefix: {@value}. */ public static final String DEFAULT_PLACEHOLDER_PREFIX = "${"; - /** Default placeholder suffix: {@value} */ + /** Default placeholder suffix: {@value}. */ public static final String DEFAULT_PLACEHOLDER_SUFFIX = "}"; - /** Default value separator: {@value} */ + /** Default value separator: {@value}. */ public static final String DEFAULT_VALUE_SEPARATOR = ":"; - /** Defaults to {@value #DEFAULT_PLACEHOLDER_PREFIX} */ + /** Defaults to {@value #DEFAULT_PLACEHOLDER_PREFIX}. */ protected String placeholderPrefix = DEFAULT_PLACEHOLDER_PREFIX; - /** Defaults to {@value #DEFAULT_PLACEHOLDER_SUFFIX} */ + /** Defaults to {@value #DEFAULT_PLACEHOLDER_SUFFIX}. */ protected String placeholderSuffix = DEFAULT_PLACEHOLDER_SUFFIX; - /** Defaults to {@value #DEFAULT_VALUE_SEPARATOR} */ + /** Defaults to {@value #DEFAULT_VALUE_SEPARATOR}. */ @Nullable protected String valueSeparator = DEFAULT_VALUE_SEPARATOR; diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/config/PropertyOverrideConfigurer.java b/spring-beans/src/main/java/org/springframework/beans/factory/config/PropertyOverrideConfigurer.java index f89593c522e..9f97840f15c 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/config/PropertyOverrideConfigurer.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/config/PropertyOverrideConfigurer.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2017 the original author or authors. + * Copyright 2002-2018 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -64,6 +64,9 @@ import org.springframework.beans.factory.BeanInitializationException; */ public class PropertyOverrideConfigurer extends PropertyResourceConfigurer { + /** + * The default bean name separator. + */ public static final String DEFAULT_BEAN_NAME_SEPARATOR = "."; @@ -72,7 +75,7 @@ public class PropertyOverrideConfigurer extends PropertyResourceConfigurer { private boolean ignoreInvalidKeys = false; /** - * Contains names of beans that have overrides + * Contains names of beans that have overrides. */ private final Set beanNames = Collections.newSetFromMap(new ConcurrentHashMap<>(16)); diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/config/YamlProcessor.java b/spring-beans/src/main/java/org/springframework/beans/factory/config/YamlProcessor.java index 5ca1a7892c0..75149c09f43 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/config/YamlProcessor.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/config/YamlProcessor.java @@ -63,7 +63,7 @@ public abstract class YamlProcessor { /** * A map of document matchers allowing callers to selectively use only * some of the documents in a YAML resource. In YAML documents are - * separated by --- lines, and each document is converted + * separated by {@code ---} lines, and each document is converted * to properties before the match is made. E.g. * 
 	 * environment: dev
@@ -343,7 +343,7 @@ public abstract class YamlProcessor {
 
 
 	/**
-	 * Status returned from {@link DocumentMatcher#matches(java.util.Properties)}
+	 * Status returned from {@link DocumentMatcher#matches(java.util.Properties)}.
 	 */
 	public enum MatchStatus {
 
diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/groovy/GroovyBeanDefinitionReader.java b/spring-beans/src/main/java/org/springframework/beans/factory/groovy/GroovyBeanDefinitionReader.java
index e9435b8349b..727920375ba 100644
--- a/spring-beans/src/main/java/org/springframework/beans/factory/groovy/GroovyBeanDefinitionReader.java
+++ b/spring-beans/src/main/java/org/springframework/beans/factory/groovy/GroovyBeanDefinitionReader.java
@@ -545,8 +545,8 @@ public class GroovyBeanDefinitionReader extends AbstractBeanDefinitionReader imp
 	}
 
 	/**
-	 * Checks whether there are any {@link RuntimeBeanReference}s inside the {@link Map}
-	 * and converts it to a {@link ManagedMap} if necessary.
+	 * Checks whether there are any {@link RuntimeBeanReference RuntimeBeanReferences}
+	 * inside the {@link Map} and converts it to a {@link ManagedMap} if necessary.
 	 * @param map the original Map
 	 * @return either the original map or a managed copy of it
 	 */
@@ -567,8 +567,8 @@ public class GroovyBeanDefinitionReader extends AbstractBeanDefinitionReader imp
 	}
 
 	/**
-	 * Checks whether there are any {@link RuntimeBeanReference}s inside the {@link List}
-	 * and converts it to a {@link ManagedList} if necessary.
+	 * Checks whether there are any {@link RuntimeBeanReference RuntimeBeanReferences}
+	 * inside the {@link List} and converts it to a {@link ManagedList} if necessary.
 	 * @param list the original List
 	 * @return either the original list or a managed copy of it
 	 */
@@ -630,7 +630,7 @@ public class GroovyBeanDefinitionReader extends AbstractBeanDefinitionReader imp
 
 	/**
 	 * This method overrides property retrieval in the scope of the
-	 * {@code GroovyBeanDefinitionReader} to either:
+	 * {@code GroovyBeanDefinitionReader}. A property retrieval will either:
 	 *