If path is null, user_path must also be null. With the current code path, the message of explicitly disabling symbolizer will never be reported. This patch adjusts the if-else structure to make that message can be reported.
[sanitizer] adjust if-else block structure to make code reachable
Thanks. I think [sanitizer] Adjust ChooseExternalSymbolizer for Windows may be more meaningful. The most significant code change is to provide a diagnostic when external_symbolizer_path= (empty). Removing an unreachable code path is a side effect.
Block2 is unreachable(user_path must not be null if want to go into block2, but that means path will never be null, which block1 will run, NOT going into block2). [...]
You can simplify this to: If path is null, user_path must also be null. ...