Parse into `Option<T>` about unhtml.rs HOT 7 CLOSED

It may be useful but currently I do not have the use case and time. Maybe you can set a "flag" as default value?

from unhtml.rs.

@Hexilee That's certainly a possibility. Would you accept a PR if I created one? I'd certainly take a look at the code if you would!

from unhtml.rs.

PR is welcome! I will take some time to review the code and make the decision.

from unhtml.rs.

@Hexilee Looking through things now; it appears as though the only two errors that can be returned are SourceNotFound and SourceEmpty. If this is correct, would you agree that the appropriate thing to do is to cast both of them to an Option?

If that's the correct course of action, I actually think this should be as simple as calling .ok() on the result, as we're disregarding both possible errors.

from unhtml.rs.

The purpose to use Result instead of Option is debugging more conveniently. Maybe you can use Option and log the err.

from unhtml.rs.

@jhpratt Option<T> is supported in v0.7, documentation will be updated in few days, you can refer to the test compound.rs now.

from unhtml.rs.

I will close this issue

from unhtml.rs.