Page MenuHomePhabricator

Improve error handling for Aphlict.
ClosedPublic

Authored by joshuaspence on Jun 7 2014, 10:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 24, 9:55 PM
Unknown Object (File)
Tue, Apr 23, 1:20 AM
Unknown Object (File)
Sun, Apr 21, 4:17 PM
Unknown Object (File)
Sat, Apr 20, 7:01 PM
Unknown Object (File)
Sun, Mar 31, 6:16 AM
Unknown Object (File)
Fri, Mar 29, 5:12 PM
Unknown Object (File)
Fri, Mar 29, 4:53 PM
Unknown Object (File)
Fri, Mar 29, 1:17 AM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP7d255aedba43: Improve error handling for Aphlict.
Summary

Currently, any error thrown when instantiating an AphlictMaster will be assumed to be due to the master already existing. This is a bit overzealous because the documentation specifically states than an ArgumentError will be throw if "the LocalConnection instance is already connected".

Test Plan

Inspected the log message.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence retitled this revision from to Improve error handling for Aphlict..
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 7 2014, 10:16 PM
epriestley updated this revision to Diff 22503.

Closed by commit rP7d255aedba43 (authored by @joshuaspence, committed by @epriestley).