Fixed a tokenization bug that caused blank lines a the console to crash horribly.

[cycript.git] / Console.cpp

diff --git a/Console.cpp b/Console.cpp
index 468a6d3d0da222a383c705f62284cd738049bb5e..c377db2abb3242ac64b4c879de99229e5568c080 100644 (file)
--- a/Console.cpp
+++ b/Console.cpp
@@ -66,10 +66,43 @@
 #include <netinet/in.h>
 #include <sys/un.h>
 
 #include <netinet/in.h>
 #include <sys/un.h>
 

+static volatile enum {
+    Working,
+    Parsing,
+    Running,
+    Sending,
+    Waiting,
+} mode_;
+

 static jmp_buf ctrlc_;
 
 static void sigint(int) {
 static jmp_buf ctrlc_;
 
 static void sigint(int) {

-    longjmp(ctrlc_, 1);
+    switch (mode_) {
+        case Working:
+            return;
+        case Parsing:
+            longjmp(ctrlc_, 1);
+        case Running:
+            throw "*** Ctrl-C";
+        case Sending:
+            return;
+        case Waiting:
+            return;
+    }
+}
+
+#if YYDEBUG
+static bool bison_;
+#endif
+static bool strict_;
+
+void Setup(CYDriver &driver, cy::parser &parser) {
+#if YYDEBUG
+    if (bison_)
+        parser.set_debug_level(1);
+#endif
+    if (strict_)
+        driver.strict_ = true;

 }
 
 void Run(int socket, const char *data, size_t size, FILE *fout = NULL, bool expand = false) {
 }
 
 void Run(int socket, const char *data, size_t size, FILE *fout = NULL, bool expand = false) {


@@ -77,12 +110,16 @@ void Run(int socket, const char *data, size_t size, FILE *fout = NULL, bool expa
 
     const char *json;
     if (socket == -1) {
 
     const char *json;
     if (socket == -1) {

+        mode_ = Running;

         json = CYExecute(pool, data);
         json = CYExecute(pool, data);

+        mode_ = Working;

         if (json != NULL)
             size = strlen(json);
     } else {
         if (json != NULL)
             size = strlen(json);
     } else {

+        mode_ = Sending;

         CYSendAll(socket, &size, sizeof(size));
         CYSendAll(socket, data, size);
         CYSendAll(socket, &size, sizeof(size));
         CYSendAll(socket, data, size);

+        mode_ = Waiting;

         CYRecvAll(socket, &size, sizeof(size));
         if (size == _not(size_t))
             json = NULL;
         CYRecvAll(socket, &size, sizeof(size));
         if (size == _not(size_t))
             json = NULL;


@@ -92,6 +129,7 @@ void Run(int socket, const char *data, size_t size, FILE *fout = NULL, bool expa
             temp[size] = '\0';
             json = temp;
         }
             temp[size] = '\0';
             json = temp;
         }

+        mode_ = Working;

     }
 
     if (json != NULL && fout != NULL) {
     }
 
     if (json != NULL && fout != NULL) {


@@ -147,13 +185,16 @@ static void Console(int socket) {
         const char *prompt("cy# ");
 
         if (setjmp(ctrlc_) != 0) {
         const char *prompt("cy# ");
 
         if (setjmp(ctrlc_) != 0) {

+            mode_ = Working;

             fputs("\n", fout);
             fflush(fout);
             goto restart;
         }
 
       read:
             fputs("\n", fout);
             fflush(fout);
             goto restart;
         }
 
       read:

+        mode_ = Parsing;

         char *line(readline(prompt));
         char *line(readline(prompt));

+        mode_ = Working;

         if (line == NULL)
             break;
 
         if (line == NULL)
             break;
 


@@ -179,8 +220,17 @@ static void Console(int socket) {
             }
         }
 
             }
         }
 

-        lines.push_back(line);

         command += line;
         command += line;

+
+        char *begin(line), *end(line + strlen(line));
+        while (char *nl = reinterpret_cast<char *>(memchr(begin, '\n', end - begin))) {
+            *nl = '\0';
+            lines.push_back(begin);
+            begin = nl + 1;
+        }
+
+        lines.push_back(begin);
+

         free(line);
 
         std::string code;
         free(line);
 
         std::string code;


@@ -190,6 +240,7 @@ static void Console(int socket) {
         else {
             CYDriver driver("");
             cy::parser parser(driver);
         else {
             CYDriver driver("");
             cy::parser parser(driver);

+            Setup(driver, parser);

 
             driver.data_ = command.c_str();
             driver.size_ = command.size();
 
             driver.data_ = command.c_str();
             driver.size_ = command.size();


@@ -197,7 +248,7 @@ static void Console(int socket) {
             if (parser.parse() != 0 || !driver.errors_.empty()) {
                 for (CYDriver::Errors::const_iterator error(driver.errors_.begin()); error != driver.errors_.end(); ++error) {
                     cy::position begin(error->location_.begin);
             if (parser.parse() != 0 || !driver.errors_.empty()) {
                 for (CYDriver::Errors::const_iterator error(driver.errors_.begin()); error != driver.errors_.end(); ++error) {
                     cy::position begin(error->location_.begin);

-                    if (begin.line != lines.size() || begin.column - 1 != lines.back().size()) {
+                    if (begin.line != lines.size() || begin.column - 1 != lines.back().size() || error->warning_) {

                         cy::position end(error->location_.end);
 
                         if (begin.line != lines.size()) {
                         cy::position end(error->location_.end);
 
                         if (begin.line != lines.size()) {


@@ -229,14 +280,15 @@ static void Console(int socket) {
                 goto read;
             }
 
                 goto read;
             }
 

-            if (driver.source_ == NULL)
+            if (driver.program_ == NULL)

                 goto restart;
 
             if (socket != -1)
                 code = command;
             else {
                 std::ostringstream str;
                 goto restart;
 
             if (socket != -1)
                 code = command;
             else {
                 std::ostringstream str;

-                driver.source_->Show(str);
+                CYOutput out(str);
+                driver.program_->Multiple(out);

                 code = str.str();
             }
         }
                 code = str.str();
             }
         }


@@ -273,14 +325,31 @@ static void *Map(const char *path, size_t *psize) {
 int main(int argc, char *argv[]) {
     bool tty(isatty(STDIN_FILENO));
     pid_t pid(_not(pid_t));
 int main(int argc, char *argv[]) {
     bool tty(isatty(STDIN_FILENO));
     pid_t pid(_not(pid_t));

+    bool compile(false);

 
 

-    for (;;) switch (getopt(argc, argv, "p:")) {
+    for (;;) switch (getopt(argc, argv, "cg:p:s")) {

         case -1:
             goto getopt;
         case '?':
         case -1:
             goto getopt;
         case '?':

-            fprintf(stderr, "usage: cycript [-p <pid>] [<script> [<arg>...]]\n");
+            fprintf(stderr, "usage: cycript [-c] [-p <pid>] [<script> [<arg>...]]\n");

             return 1;
 
             return 1;
 

+        case 'c':
+            compile = true;
+        break;
+
+        case 'g':
+            if (false);
+#if YYDEBUG
+            else if (strcmp(optarg, "bison") == 0)
+                bison_ = true;
+#endif
+            else {
+                fprintf(stderr, "invalid name for -g\n");
+                return 1;
+            }
+        break;
+

         case 'p': {
             size_t size(strlen(optarg));
             char *end;
         case 'p': {
             size_t size(strlen(optarg));
             char *end;


@@ -290,15 +359,24 @@ int main(int argc, char *argv[]) {
                 return 1;
             }
         } break;
                 return 1;
             }
         } break;

+
+        case 's':
+            strict_ = true;
+        break;

     } getopt:;
 
     const char *script;
 
     } getopt:;
 
     const char *script;
 

-    if (optind < argc - 1 && pid != _not(pid_t)) {
+    if (pid != _not(pid_t) && optind < argc - 1) {

         fprintf(stderr, "-p cannot set argv\n");
         return 1;
     }
 
         fprintf(stderr, "-p cannot set argv\n");
         return 1;
     }
 

+    if (pid != _not(pid_t) && compile) {
+        fprintf(stderr, "-p conflicts with -c\n");
+        return 1;
+    }
+

     if (optind == argc)
         script = NULL;
     else {
     if (optind == argc)
         script = NULL;
     else {


@@ -334,6 +412,7 @@ int main(int argc, char *argv[]) {
     else {
         CYDriver driver(script ?: "<stdin>");
         cy::parser parser(driver);
     else {
         CYDriver driver(script ?: "<stdin>");
         cy::parser parser(driver);

+        Setup(driver, parser);

 
         char *start, *end;
 
 
         char *start, *end;
 


@@ -363,14 +442,18 @@ int main(int argc, char *argv[]) {
         if (parser.parse() != 0 || !driver.errors_.empty()) {
             for (CYDriver::Errors::const_iterator i(driver.errors_.begin()); i != driver.errors_.end(); ++i)
                 std::cerr << i->location_.begin << ": " << i->message_ << std::endl;
         if (parser.parse() != 0 || !driver.errors_.empty()) {
             for (CYDriver::Errors::const_iterator i(driver.errors_.begin()); i != driver.errors_.end(); ++i)
                 std::cerr << i->location_.begin << ": " << i->message_ << std::endl;

-        } else if (driver.source_ != NULL)
+        } else if (driver.program_ != NULL)

             if (socket != -1)
                 Run(socket, start, end - start, stdout);
             else {
                 std::ostringstream str;
             if (socket != -1)
                 Run(socket, start, end - start, stdout);
             else {
                 std::ostringstream str;

-                driver.source_->Show(str);
+                CYOutput out(str);
+                driver.program_->Multiple(out);

                 std::string code(str.str());
                 std::string code(str.str());

-                Run(socket, code);
+                if (compile)
+                    std::cout << code;
+                else
+                    Run(socket, code);

             }
     }
 
             }
     }