| Index: tools/gn/input_file_manager.cc
|
| diff --git a/tools/gn/input_file_manager.cc b/tools/gn/input_file_manager.cc
|
| index f358fdc2e68420b19f9e65b6c34ab6f38560953e..eb255080a9da68445cec8b3d03e4c612a7b6ae3a 100644
|
| --- a/tools/gn/input_file_manager.cc
|
| +++ b/tools/gn/input_file_manager.cc
|
| @@ -215,20 +215,11 @@
|
| }
|
| }
|
|
|
| - // The other load could have failed. In this case that error was probably
|
| - // printed to the console, but we need to return something here, so make up a
|
| + // The other load could have failed. In this case that error will be printed
|
| + // to the console, but we need to return something here, so make up a
|
| // dummy error.
|
| - //
|
| - // There is a race condition. The other load could have failed, but if the
|
| - // other thread is delayed for some reason, this thread could end up
|
| - // reporting the error to the scheduler first (since first error report
|
| - // wins). The user will see this one and the "real" one will be discarded.
|
| - if (!data->parsed_root) {
|
| - *err = Err(origin, "File parse failed.",
|
| - "If you see this, I'm really sorry, but a race condition has caused\n"
|
| - "me to eat your error message. It was crunchy. If the parse error\n"
|
| - "in your imported file isn't obvious, try re-running GN.");
|
| - }
|
| + if (!data->parsed_root)
|
| + *err = Err(origin, "File parse failed");
|
| return data->parsed_root.get();
|
| }
|
|
|
|
|