commit 39f6da479392bc9fece31618fb138ca84ccf85be
author Ethan Nicholas <ethannicholas@google.com> Mon Aug 23 13:10:07 2021 -0400
committer SkCQ <skcq-be@skia-corp.google.com.iam.gserviceaccount.com> Mon Aug 23 18:40:09 2021 +0000
tree 3d3c0ff4c2e9863c0779b0b1d6e062c51d443ac0
parent 405eb7b72fe143222265840cd36a7f77d52bf747

Switched Context::errors() over to a field access

Now that Context owns the errors pointer, there is no magic behind its
accessor and no reason not to just use a straight field access.

Change-Id: I3f771f458ffdaf95d6289ba5767535a78126cc0b
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/441312
Reviewed-by: John Stiles <johnstiles@google.com>
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>

src/sksl/SkSLIRGenerator.cpp

diff --git a/src/sksl/SkSLIRGenerator.cpp b/src/sksl/SkSLIRGenerator.cpp
index 73278ca..79f9b26 100644
--- a/src/sksl/SkSLIRGenerator.cpp
+++ b/src/sksl/SkSLIRGenerator.cpp
@@ -722,7 +722,7 @@
     for (const auto& f : kModifierFlags) {
         if (modifierFlags & f.flag) {
             if (!(permittedModifierFlags & f.flag)) {
-                context.errors().error(offset, "'" + String(f.name) + "' is not permitted here");
+                context.fErrors->error(offset, "'" + String(f.name) + "' is not permitted here");
             }
             modifierFlags &= ~f.flag;
         }
@@ -750,7 +750,7 @@
     for (const auto& lf : kLayoutFlags) {
         if (layoutFlags & lf.flag) {
             if (!(permittedLayoutFlags & lf.flag)) {
-                context.errors().error(
+                context.fErrors->error(
                         offset, "layout qualifier '" + String(lf.name) + "' is not permitted here");
             }
             layoutFlags &= ~lf.flag;