[master] ServiceLocator.kt redundant code ?

In [app/src/mock/java/com/example/android/architecture/blueprints/todoapp/ServiceLocator.kt](https://github.com/googlesamples/android-architecture/blob/master/app/src/mock/java/com/example/android/architecture/blueprints/todoapp/ServiceLocator.kt)，L43. Why repeat `tasksRepository ?:` in return?

Can the return be simplified as `return tasksRepository ?: createTasksRepository(context)` ?

```kotlin
fun provideTasksRepository(context: Context): TasksRepository {
synchronized(this) {
return tasksRepository ?: tasksRepository ?: createTasksRepository(context) // HERE!
}
}
```

Thanks for your attention.

Open app/src/mock/java/com/example/android/architecture/blueprints/todoapp/ServiceLocator.kt and inspect line 43 in provideTasksRepository. Confirm whether the repeated tasksRepository ?: is redundant, then simplify the return expression if appropriate; the work is done when the duplicate expression is removed without changing repository creation behavior.

Written by the indexing model from the issue text.