inkle / inkle/ink

Don't throw exception from ContentAtPath

Open
#211 0 comments 0 reactions 0 assignees View on GitHub
enhancement help wanted
Dominant language
C#
Stars
4.9k
Forks
540
PR merge metrics
No merged PRs in 30d

Description

`Container`'s `ContentAtPath` currently throws an exception if it fails, which creates some rather ugly `try/catch` code in a number of places where we want to be able to fail gracefully if the content can't be found.

Convert `ContentAtPath` to return null if it doesn't find the content, and throw exceptions at the call site where it's still intended. (Such as in the main `Step()` function).

Note that this should be relatively straightforward if anyone wants to give it a try! (Though not before #210 is fixed so you can verify your code!)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.