From eeafbe31a6455d9f13d5b3719b5db6c6ab0562b1 Mon Sep 17 00:00:00 2001
From: "igor%mir2.org" <igor%mir2.org@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Wed, 9 Jul 2003 19:27:46 +0000
Subject: [PATCH] Throw IllegalArgumentException from
 compileString/compileReader if script line number is negative instead of
 catching it much later via Context.codeBug().

git-svn-id: svn://10.0.0.236/trunk@144650 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/js/rhino/src/org/mozilla/javascript/Context.java | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/mozilla/js/rhino/src/org/mozilla/javascript/Context.java b/mozilla/js/rhino/src/org/mozilla/javascript/Context.java
index 0b87dd55a26..e6edfaa4f59 100644
--- a/mozilla/js/rhino/src/org/mozilla/javascript/Context.java
+++ b/mozilla/js/rhino/src/org/mozilla/javascript/Context.java
@@ -897,6 +897,10 @@ public class Context {
                                 int lineno, Object securityDomain)
         throws IOException
     {
+        if (lineno < 0) {
+            throw new IllegalArgumentException(
+                "Line number can not be negative:"+lineno);
+        }
         return (Script) compile(scope, in, null, sourceName, lineno,
                                 securityDomain, false);
     }
@@ -923,6 +927,10 @@ public class Context {
                                 String sourceName, int lineno,
                                 Object securityDomain)
     {
+        if (lineno < 0) {
+            throw new IllegalArgumentException(
+                "Line number can not be negative:"+lineno);
+        }
         try {
             return (Script) compile(scope, null, source, sourceName, lineno,
                                     securityDomain, false);