Skip to content

Commit 14c6485

Browse files
config: Remove unnecessary nil checks
Signed-off-by: Daniel Kleinstein <daniel.kleinstein@gmail.com>
1 parent e5f49f9 commit 14c6485

File tree

2 files changed

+3
-7
lines changed

2 files changed

+3
-7
lines changed

api/config/v1/config.go

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -96,13 +96,9 @@ func DisableResourceNamingInConfig(logger logger, config *Config) {
9696
config.Resources.MIGs = nil
9797

9898
// Disable renaming / device selection in Sharing.TimeSlicing.Resources
99-
if (config.sharing.TimeSlicing != nil) {
100-
config.Sharing.TimeSlicing.disableResoureRenaming(logger, "timeSlicing")
101-
}
99+
config.Sharing.TimeSlicing.disableResoureRenaming(logger, "timeSlicing")
102100
// Disable renaming / device selection in Sharing.MPS.Resources
103-
if (config.Sharing.MPS != nil) {
104-
config.Sharing.MPS.disableResoureRenaming(logger, "mps")
105-
}
101+
config.Sharing.MPS.disableResoureRenaming(logger, "mps")
106102
}
107103

108104
// parseConfig parses a config file as either YAML of JSON and unmarshals it into a Config struct.

api/config/v1/sharing.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ func (s *Sharing) SharingStrategy() SharingStrategy {
3838
return SharingStrategyMPS
3939
}
4040

41-
if s.TimeSlicing != nil && s.TimeSlicing.isReplicated() {
41+
if s.TimeSlicing.isReplicated() {
4242
return SharingStrategyTimeSlicing
4343
}
4444
return SharingStrategyNone

0 commit comments

Comments
 (0)