AK: Rename the local variable in the TRY() macro to avoid name clashes

"result" is a tad bit too generic to provide a clash-free experience -
we found instances in LibJS where this breaks already. Essentially this
doesn't work:

    auto foo = TRY(bar(result));

Because it expands to the following within the TRY() scope:

    {
        auto result = bar(result);
        ...
    }

And that of course fails:

    error: use of ‘result’ before deduction of ‘auto’

The simple solution here is to use a name that is much less likely to
clash with anything used in the expression ("_temporary_result"). :^)

1 files changed, 6 insertions, 6 deletions

diff --git a/AK/Try.h b/AK/Try.h
index 862fc7dbc5..4efb21a5a0 100644
--- a/AK/Try.h
+++ b/AK/Try.h
@@ -9,10 +9,10 @@
 // NOTE: This macro works with any result type that has the expected APIs.
 //       It's designed with AK::Result and Kernel::KResult in mind.
 
-#define TRY(expression)                    \
-    ({                                     \
-        auto result = (expression);        \
-        if (result.is_error())             \
-            return result.release_error(); \
-        result.release_value();            \
+#define TRY(expression)                               \
+    ({                                                \
+        auto _temporary_result = (expression);        \
+        if (_temporary_result.is_error())             \
+            return _temporary_result.release_error(); \
+        _temporary_result.release_value();            \
     })