Wrap run.php with try/catch block and ...

Brought to you by: hkodungallur, tchule

#63 Wrap run.php with try/catch block and ...

Status: Fixed

Owner: nobody

Labels: None

Priority: Medium

Type: Defect

Updated: 2013-12-02

Created: 2013-11-11

Creator: Anonymous

Private: No

Originally created by: kbcm...@gmail.com

While running run.php against code that had an unterminated comment, run.php had a stack trace but exited with a 0 return code. This made detecting the problem more difficult but I believe all programs should handle uncaught exceptions more gracefully.

Related

Discussion

Comment has been marked as spam.
Undo

View and moderate all "tickets Discussion" comments posted by this user

Mark all as spam, and block user from posting to "Tickets"

Anonymous - 2013-11-11

Originally posted by: tch...@hotmail.com

(No comment was entered for this change.)

Status: Accepted

*Originally posted by:* [tch...@hotmail.com](http://code.google.com/u/103717617877343775305/) *\(No comment was entered for this change.\)* **Status:** Accepted

Add attachments
Cancel
You seem to have CSS turned off. Please don't fill out this field.

You seem to have CSS turned off. Please don't fill out this field.

New Attachment:

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Comment has been marked as spam.
Undo

View and moderate all "tickets Discussion" comments posted by this user

Mark all as spam, and block user from posting to "Tickets"

Anonymous - 2013-11-25

Originally posted by: tch...@hotmail.com

Yes, should catch the exception, report a problem (with the exception message) and continue to the next file.

*Originally posted by:* [tch...@hotmail.com](http://code.google.com/u/103717617877343775305/) Yes, should catch the exception, report a problem \(with the exception message\) and continue to the next file.

Add attachments
Cancel
You seem to have CSS turned off. Please don't fill out this field.

You seem to have CSS turned off. Please don't fill out this field.

New Attachment:

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Comment has been marked as spam.
Undo

View and moderate all "tickets Discussion" comments posted by this user

Mark all as spam, and block user from posting to "Tickets"

Anonymous - 2013-12-02

Originally posted by: tch...@hotmail.com

OK, Error management in PHP is a mess.
Parsing errors are not catchable and are not calling error handlers.
Used a trick with error_get_last to test for parsing errors : http://php.net/manual/en/function.error-get-last.php

Status: Fixed

*Originally posted by:* [tch...@hotmail.com](http://code.google.com/u/103717617877343775305/) OK, Error management in PHP is a mess. Parsing errors are not catchable and are not calling error handlers. Used a trick with error\_get\_last to test for parsing errors : [http://php.net/manual/en/function.error-get-last.php](http://php.net/manual/en/function.error-get-last.php) **Status:** Fixed

Add attachments
Cancel
You seem to have CSS turned off. Please don't fill out this field.

You seem to have CSS turned off. Please don't fill out this field.

New Attachment:

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.