Add strict option to ParseExpression (#410)

sebastienros · Sep 10, 2023 · 5713567 · 5713567
1 parent 423d691
commit 5713567
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/src/Esprima/Ast/Expression.cs b/src/Esprima/Ast/Expression.cs
@@ -13,7 +13,7 @@ protected Expression(Nodes type) : base(type)
 
     /// <summary>
     /// Gets or sets the list of tokens associated with the AST represented by this node.
-    /// This property is automatically set by <see cref="JavaScriptParser.ParseExpression(string)"/> when <see cref="ParserOptions.Tokens"/> is set to <see langword="true"/>.
+    /// This property is automatically set by <see cref="JavaScriptParser.ParseExpression(string, bool)"/> when <see cref="ParserOptions.Tokens"/> is set to <see langword="true"/>.
     /// </summary>
     /// <remarks>
     /// The operation is not guaranteed to be thread-safe. In case concurrent access or update is possible, the necessary synchronization is caller's responsibility.
@@ -28,7 +28,7 @@ public IReadOnlyList<SyntaxToken>? Tokens
 
     /// <summary>
     /// Gets or sets the list of comments associated with the AST represented by this node.
-    /// This property is automatically set by <see cref="JavaScriptParser.ParseExpression(string)"/> when <see cref="ParserOptions.Comments"/> is set to <see langword="true"/>.
+    /// This property is automatically set by <see cref="JavaScriptParser.ParseExpression(string, bool)"/> when <see cref="ParserOptions.Comments"/> is set to <see langword="true"/>.
     /// </summary>
     /// <remarks>
     /// The operation is not guaranteed to be thread-safe. In case concurrent access or update is possible, the necessary synchronization is caller's responsibility.

diff --git a/src/Esprima/JavaScriptParser.cs b/src/Esprima/JavaScriptParser.cs
@@ -8,7 +8,7 @@ namespace Esprima;
 /// Provides JavaScript parsing capabilities.
 /// </summary>
 /// <remarks>
-/// Use the <see cref="ParseScript" />, <see cref="ParseModule" /> or <see cref="ParseExpression(string)" /> methods to parse the JavaScript code.
+/// Use the <see cref="ParseScript" />, <see cref="ParseModule" /> or <see cref="ParseExpression(string, bool)" /> methods to parse the JavaScript code.
 /// </remarks>
 public partial class JavaScriptParser
 {
@@ -2751,11 +2751,12 @@ private Expression ParseExpression()
     /// <summary>
     /// Parses the code as a JavaScript expression.
     /// </summary>
-    public Expression ParseExpression(string code)
+    public Expression ParseExpression(string code, bool strict = false)
     {
         Reset(code, source: null);
         try
         {
+            _context.Strict = strict;
             _context.IsAsync = true;
             return FinalizeRoot(ParseExpression());
         }

diff --git a/src/Esprima/JsxParser.cs b/src/Esprima/JsxParser.cs
@@ -9,7 +9,7 @@ namespace Esprima;
 /// </summary>
 /// <remarks>
 /// Use the <see cref="JavaScriptParser.ParseScript" />, <see cref="JavaScriptParser.ParseModule" /> or
-/// <see cref="JavaScriptParser.ParseExpression(string)" /> methods to parse the JSX code.
+/// <see cref="JavaScriptParser.ParseExpression(string, bool)" /> methods to parse the JSX code.
 /// </remarks>
 public class JsxParser : JavaScriptParser
 {