Skip to content

Commit 9f1d851

Browse files
committed
Polish Kotlin source code style
1 parent 5843173 commit 9f1d851

File tree

27 files changed

+130
-91
lines changed

27 files changed

+130
-91
lines changed

spring-beans/src/main/kotlin/org/springframework/beans/factory/BeanFactoryExtensions.kt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616

1717
package org.springframework.beans.factory
1818

19-
2019
/**
2120
* Extension for [BeanFactory.getBean] providing a `getBean<Foo>()` variant.
2221
*

spring-beans/src/main/kotlin/org/springframework/beans/factory/ListableBeanFactoryExtensions.kt

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -17,25 +17,29 @@
1717
package org.springframework.beans.factory
1818

1919
/**
20-
* Extension for [ListableBeanFactory.getBeanNamesForType] providing a `getBeanNamesForType<Foo>()` variant.
20+
* Extension for [ListableBeanFactory.getBeanNamesForType] providing a
21+
* `getBeanNamesForType<Foo>()` variant.
2122
*
2223
* @author Sebastien Deleuze
2324
* @since 5.0
2425
*/
25-
inline fun <reified T : Any> ListableBeanFactory.getBeanNamesForType(includeNonSingletons: Boolean = true, allowEagerInit: Boolean = true): Array<out String> =
26-
getBeanNamesForType(T::class.java, includeNonSingletons, allowEagerInit)
26+
inline fun <reified T : Any> ListableBeanFactory.getBeanNamesForType(includeNonSingletons: Boolean = true,
27+
allowEagerInit: Boolean = true): Array<out String> =
28+
getBeanNamesForType(T::class.java, includeNonSingletons, allowEagerInit)
2729

2830
/**
2931
* Extension for [ListableBeanFactory.getBeansOfType] providing a `getBeansOfType<Foo>()` variant.
3032
*
3133
* @author Sebastien Deleuze
3234
* @since 5.0
3335
*/
34-
inline fun <reified T : Any> ListableBeanFactory.getBeansOfType(includeNonSingletons: Boolean = true, allowEagerInit: Boolean = true): Map<String, T> =
35-
getBeansOfType(T::class.java, includeNonSingletons, allowEagerInit)
36+
inline fun <reified T : Any> ListableBeanFactory.getBeansOfType(includeNonSingletons: Boolean = true,
37+
allowEagerInit: Boolean = true): Map<String, T> =
38+
getBeansOfType(T::class.java, includeNonSingletons, allowEagerInit)
3639

3740
/**
38-
* Extension for [ListableBeanFactory.getBeanNamesForAnnotation] providing a `getBeansOfType<Foo>()` variant.
41+
* Extension for [ListableBeanFactory.getBeanNamesForAnnotation] providing a
42+
* `getBeansOfType<Foo>()` variant.
3943
*
4044
* @author Sebastien Deleuze
4145
* @since 5.0
@@ -44,7 +48,8 @@ inline fun <reified T : Annotation> ListableBeanFactory.getBeanNamesForAnnotatio
4448
getBeanNamesForAnnotation(T::class.java)
4549

4650
/**
47-
* Extension for [ListableBeanFactory.getBeansWithAnnotation] providing a `getBeansWithAnnotation<Foo>()` variant.
51+
* Extension for [ListableBeanFactory.getBeansWithAnnotation] providing a
52+
* `getBeansWithAnnotation<Foo>()` variant.
4853
*
4954
* @author Sebastien Deleuze
5055
* @since 5.0
@@ -53,7 +58,8 @@ inline fun <reified T : Annotation> ListableBeanFactory.getBeansWithAnnotation()
5358
getBeansWithAnnotation(T::class.java)
5459

5560
/**
56-
* Extension for [ListableBeanFactory.findAnnotationOnBean] providing a `findAnnotationOnBean<Foo>("foo")` variant.
61+
* Extension for [ListableBeanFactory.findAnnotationOnBean] providing a
62+
* `findAnnotationOnBean<Foo>("foo")` variant.
5763
*
5864
* @author Sebastien Deleuze
5965
* @since 5.0

spring-beans/src/test/kotlin/org/springframework/beans/BeanUtilsKotlinTests.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ import org.junit.Assert.*
2020
import org.junit.Test
2121

2222
/**
23-
* Kotlin tests for {@link BeanUtils}
23+
* Kotlin tests for {@link BeanUtils}.
2424
*
2525
* @author Sebastien Deleuze
2626
*/

spring-beans/src/test/kotlin/org/springframework/beans/factory/BeanFactoryExtensionsTests.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ import org.mockito.Mockito.*
2424
import org.mockito.junit.MockitoJUnitRunner
2525

2626
/**
27-
* Mock object based tests for BeanFactory Kotlin extensions
27+
* Mock object based tests for BeanFactory Kotlin extensions.
2828
*
2929
* @author Sebastien Deleuze
3030
*/

spring-beans/src/test/kotlin/org/springframework/beans/factory/annotation/KotlinAutowiredTests.kt

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,8 @@ class KotlinAutowiredTests {
231231
val optional: String = "foo",
232232
val injectedFromConstructor: TestBean
233233
) {
234-
@Autowired constructor(injectedFromSecondaryConstructor: Colour, injectedFromConstructor: TestBean, optional: String = "bar") : this(optional, injectedFromConstructor) {
234+
@Autowired constructor(injectedFromSecondaryConstructor: Colour, injectedFromConstructor: TestBean,
235+
optional: String = "bar") : this(optional, injectedFromConstructor) {
235236
this.injectedFromSecondaryConstructor = injectedFromSecondaryConstructor
236237
}
237238

@@ -247,7 +248,8 @@ class KotlinAutowiredTests {
247248
val optional: String = "foo",
248249
val injectedFromConstructor: TestBean
249250
) {
250-
constructor(injectedFromSecondaryConstructor: Colour, injectedFromConstructor: TestBean, optional: String = "bar") : this(optional, injectedFromConstructor) {
251+
constructor(injectedFromSecondaryConstructor: Colour, injectedFromConstructor: TestBean,
252+
optional: String = "bar") : this(optional, injectedFromConstructor) {
251253
this.injectedFromSecondaryConstructor = injectedFromSecondaryConstructor
252254
}
253255

spring-context/src/main/kotlin/org/springframework/context/support/BeanDefinitionDsl.kt

Lines changed: 18 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -71,11 +71,13 @@ fun beans(init: BeanDefinitionDsl.() -> Unit): BeanDefinitionDsl {
7171
* Class implementing functional bean definition Kotlin DSL.
7272
*
7373
* @constructor Create a new bean definition DSL.
74-
* @param condition the predicate to fulfill in order to take in account the inner bean definition block
74+
* @param condition the predicate to fulfill in order to take in account the inner
75+
* bean definition block
7576
* @author Sebastien Deleuze
7677
* @since 5.0
7778
*/
78-
open class BeanDefinitionDsl(private val condition: (ConfigurableEnvironment) -> Boolean = { true }) : ApplicationContextInitializer<GenericApplicationContext> {
79+
open class BeanDefinitionDsl(private val condition: (ConfigurableEnvironment) -> Boolean = { true })
80+
: ApplicationContextInitializer<GenericApplicationContext> {
7981

8082
@PublishedApi
8183
internal val registrations = arrayListOf<(GenericApplicationContext) -> Unit>()
@@ -131,7 +133,8 @@ open class BeanDefinitionDsl(private val condition: (ConfigurableEnvironment) ->
131133
/**
132134
* Provide read access to some application context facilities.
133135
* @constructor Create a new bean definition context.
134-
* @param context the `ApplicationContext` instance to use for retrieving bean references, `Environment`, etc.
136+
* @param context the `ApplicationContext` instance to use for retrieving bean
137+
* references, `Environment`, etc.
135138
*/
136139
inner class BeanDefinitionContext(@PublishedApi internal val context: GenericApplicationContext) {
137140

@@ -163,7 +166,8 @@ open class BeanDefinitionDsl(private val condition: (ConfigurableEnvironment) ->
163166
* @param isLazyInit Set whether this bean should be lazily initialized.
164167
* @param isPrimary Set whether this bean is a primary autowire candidate.
165168
* @param autowireMode Set the autowire mode, `Autowire.CONSTRUCTOR` by default
166-
* @param isAutowireCandidate Set whether this bean is a candidate for getting autowired into some other bean.
169+
* @param isAutowireCandidate Set whether this bean is a candidate for getting
170+
* autowired into some other bean.
167171
* @see GenericApplicationContext.registerBean
168172
* @see org.springframework.beans.factory.config.BeanDefinition
169173
*/
@@ -200,7 +204,8 @@ open class BeanDefinitionDsl(private val condition: (ConfigurableEnvironment) ->
200204
* @param isLazyInit Set whether this bean should be lazily initialized.
201205
* @param isPrimary Set whether this bean is a primary autowire candidate.
202206
* @param autowireMode Set the autowire mode, `Autowire.NO` by default
203-
* @param isAutowireCandidate Set whether this bean is a candidate for getting autowired into some other bean.
207+
* @param isAutowireCandidate Set whether this bean is a candidate for getting
208+
* autowired into some other bean.
204209
* @param function the bean supplier function
205210
* @see GenericApplicationContext.registerBean
206211
* @see org.springframework.beans.factory.config.BeanDefinition
@@ -226,8 +231,10 @@ open class BeanDefinitionDsl(private val condition: (ConfigurableEnvironment) ->
226231
registrations.add {
227232
val beanContext = BeanDefinitionContext(it)
228233
when (name) {
229-
null -> it.registerBean(T::class.java, Supplier { function.invoke(beanContext) }, customizer)
230-
else -> it.registerBean(name, T::class.java, Supplier { function.invoke(beanContext) }, customizer)
234+
null -> it.registerBean(T::class.java,
235+
Supplier { function.invoke(beanContext) }, customizer)
236+
else -> it.registerBean(name, T::class.java,
237+
Supplier { function.invoke(beanContext) }, customizer)
231238
}
232239
}
233240
}
@@ -246,9 +253,11 @@ open class BeanDefinitionDsl(private val condition: (ConfigurableEnvironment) ->
246253
/**
247254
* Take in account bean definitions enclosed in the provided lambda only when the
248255
* specified environment-based predicate is true.
249-
* @param condition the predicate to fulfill in order to take in account the inner bean definition block
256+
* @param condition the predicate to fulfill in order to take in account the inner
257+
* bean definition block
250258
*/
251-
fun environment(condition: ConfigurableEnvironment.() -> Boolean, init: BeanDefinitionDsl.() -> Unit): BeanDefinitionDsl {
259+
fun environment(condition: ConfigurableEnvironment.() -> Boolean,
260+
init: BeanDefinitionDsl.() -> Unit): BeanDefinitionDsl {
252261
val beans = BeanDefinitionDsl(condition::invoke)
253262
beans.init()
254263
children.add(beans)

spring-context/src/main/kotlin/org/springframework/context/support/GenericApplicationContextExtensions.kt

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,9 @@ import org.springframework.beans.factory.config.BeanDefinitionCustomizer
2020
import org.springframework.context.ApplicationContext
2121
import java.util.function.Supplier
2222

23-
2423
/**
25-
* Extension for [GenericApplicationContext.registerBean] providing a `registerBean<Foo>()` variant.
24+
* Extension for [GenericApplicationContext.registerBean] providing a
25+
* `registerBean<Foo>()` variant.
2626
*
2727
* @author Sebastien Deleuze
2828
* @since 5.0
@@ -32,12 +32,14 @@ inline fun <reified T : Any> GenericApplicationContext.registerBean(vararg custo
3232
}
3333

3434
/**
35-
* Extension for [GenericApplicationContext.registerBean] providing a `registerBean<Foo>("foo")` variant.
35+
* Extension for [GenericApplicationContext.registerBean] providing a
36+
* `registerBean<Foo>("foo")` variant.
3637
*
3738
* @author Sebastien Deleuze
3839
* @since 5.0
3940
*/
40-
inline fun <reified T : Any> GenericApplicationContext.registerBean(beanName: String, vararg customizers: BeanDefinitionCustomizer) {
41+
inline fun <reified T : Any> GenericApplicationContext.registerBean(beanName: String,
42+
vararg customizers: BeanDefinitionCustomizer) {
4143
registerBean(beanName, T::class.java, *customizers)
4244
}
4345

@@ -53,7 +55,8 @@ inline fun <reified T : Any> GenericApplicationContext.registerBean(
5355
}
5456

5557
/**
56-
* Extension for [GenericApplicationContext.registerBean] providing a `registerBean("foo") { Foo() }` variant.
58+
* Extension for [GenericApplicationContext.registerBean] providing a
59+
* `registerBean("foo") { Foo() }` variant.
5760
*
5861
* @author Sebastien Deleuze
5962
* @since 5.0
@@ -64,10 +67,12 @@ inline fun <reified T : Any> GenericApplicationContext.registerBean(name: String
6467
}
6568

6669
/**
67-
* Extension for [GenericApplicationContext] allowing `GenericApplicationContext { ... }` style initialization.
70+
* Extension for [GenericApplicationContext] allowing `GenericApplicationContext { ... }`
71+
* style initialization.
6872
*
6973
* @author Sebastien Deleuze
7074
* @since 5.0
7175
*/
72-
fun GenericApplicationContext(configure: GenericApplicationContext.() -> Unit) = GenericApplicationContext().apply(configure)
76+
fun GenericApplicationContext(configure: GenericApplicationContext.() -> Unit) =
77+
GenericApplicationContext().apply(configure)
7378

spring-context/src/test/kotlin/org/springframework/context/annotation/AnnotationConfigApplicationContextExtensionsTests.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ import org.springframework.beans.factory.getBean
2222
import org.springframework.context.support.registerBean
2323

2424
/**
25-
* Tests for [AnnotationConfigApplicationContext] Kotlin extensions
25+
* Tests for [AnnotationConfigApplicationContext] Kotlin extensions.
2626
*
2727
* @author Sebastien Deleuze
2828
*/

spring-context/src/test/kotlin/org/springframework/ui/ModelExtensionsTests.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,8 @@ import org.junit.Assert.assertEquals
2020
import org.junit.Assert.assertTrue
2121
import org.junit.Test
2222

23-
2423
/**
25-
* Tests for [Model] Kotlin extensions
24+
* Tests for [Model] Kotlin extensions.
2625
*
2726
* @author Sebastien Deleuze
2827
*/

spring-context/src/test/kotlin/org/springframework/ui/ModelMapExtensionsTests.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,8 @@ import org.junit.Assert.assertEquals
2020
import org.junit.Assert.assertTrue
2121
import org.junit.Test
2222

23-
2423
/**
25-
* Tests for [ModelMap] Kotlin extensions
24+
* Tests for [ModelMap] Kotlin extensions.
2625
*
2726
* @author Sebastien Deleuze
2827
*/

0 commit comments

Comments
 (0)