Dereference optional when checking boolean flag.

Tensorstore Team · copybara-github · commit 28fb969421ef · 2025-09-19T15:08:48.000-07:00
PiperOrigin-RevId: 809204710
Change-Id: Iaddd8165768859268c3e409c8ffe5f87b52b7a11
diff --git a/tensorstore/kvstore/gcs_grpc/default_endpoint.cc b/tensorstore/kvstore/gcs_grpc/default_endpoint.cc
@@ -42,7 +42,7 @@ bool UseDirectPathGcsEndpointByDefaultImpl() {
   if (auto disable_direct_path =
           internal::GetFlagOrEnvValue(FLAGS_tensorstore_disable_direct_path,
                                       "GOOGLE_CLOUD_DISABLE_DIRECT_PATH");
-      disable_direct_path.has_value() && disable_direct_path) {
+      disable_direct_path.has_value() && *disable_direct_path) {
     return false;
   }
 

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ bool UseDirectPathGcsEndpointByDefaultImpl() {`
`42`	`42`	`if (auto disable_direct_path =`
`43`	`43`	`internal::GetFlagOrEnvValue(FLAGS_tensorstore_disable_direct_path,`
`44`	`44`	`"GOOGLE_CLOUD_DISABLE_DIRECT_PATH");`
`45`		`- disable_direct_path.has_value() && disable_direct_path) {`
	`45`	`+ disable_direct_path.has_value() && *disable_direct_path) {`
`46`	`46`	`return false;`
`47`	`47`	`}`
`48`	`48`