MTC created this revision.
MTC added reviewers: NoQ, george.karpenkov.
Herald added subscribers: mikhail.ramalho, a.sidorin, szepet, xazax.hun.

The typo of the description for default bindings can be confusing.


Repository:
  rC Clang

https://reviews.llvm.org/D50382

Files:
  docs/analyzer/RegionStore.txt


Index: docs/analyzer/RegionStore.txt
===================================================================
--- docs/analyzer/RegionStore.txt
+++ docs/analyzer/RegionStore.txt
@@ -118,7 +118,7 @@
   int manyInts[10];
   manyInts[1] = 42;   // Creates a Direct binding for manyInts[1].
   print(manyInts[1]); // Retrieves the Direct binding for manyInts[1];
-  print(manyInts[0]); // There is no Direct binding for manyInts[1].
+  print(manyInts[0]); // There is no Direct binding for manyInts[0].
                       // Is there a Default binding for the entire array?
                       // There is not, but it is a stack variable, so we use
                       // "uninitialized" as the default value (and emit a
@@ -166,6 +166,6 @@
     return p2.x;    // The binding for FieldRegion 'p2.x' is requested.
                     // There is no Direct binding, so we look for a Default
                     // binding to 'p2' and find the LCV.
-                    // Because it's an LCV, we look at our requested region
+                    // Because it's a LCV, we look at our requested region
                     // and see that it's the '.x' field. We ask for the value
                     // of 'p.x' within the snapshot, and get back 42.


Index: docs/analyzer/RegionStore.txt
===================================================================
--- docs/analyzer/RegionStore.txt
+++ docs/analyzer/RegionStore.txt
@@ -118,7 +118,7 @@
   int manyInts[10];
   manyInts[1] = 42;   // Creates a Direct binding for manyInts[1].
   print(manyInts[1]); // Retrieves the Direct binding for manyInts[1];
-  print(manyInts[0]); // There is no Direct binding for manyInts[1].
+  print(manyInts[0]); // There is no Direct binding for manyInts[0].
                       // Is there a Default binding for the entire array?
                       // There is not, but it is a stack variable, so we use
                       // "uninitialized" as the default value (and emit a
@@ -166,6 +166,6 @@
     return p2.x;    // The binding for FieldRegion 'p2.x' is requested.
                     // There is no Direct binding, so we look for a Default
                     // binding to 'p2' and find the LCV.
-                    // Because it's an LCV, we look at our requested region
+                    // Because it's a LCV, we look at our requested region
                     // and see that it's the '.x' field. We ask for the value
                     // of 'p.x' within the snapshot, and get back 42.
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to