Skip to content

Conversation

@lbargaoanu
Copy link
Contributor

It's true that anybody can write this, but I thought it might be nice to have something built in.

@jbogard
Copy link
Contributor

jbogard commented Jul 30, 2016

What's the difference between this and a type converter?

@lbargaoanu
Copy link
Contributor Author

IsMatch is more flexible. But anyway, what's the difference in general? I wasn't sure whether to add the extra parameters so I didn't.

@lbargaoanu
Copy link
Contributor Author

I think it didn't feel like an interface with all those parameters, but they are input, so I guess they do no harm. The main point was that probably no one would implement the interface as it is. But if it still feels redundant, just close it.

@jbogard jbogard added the Feature label Aug 1, 2016
@jbogard jbogard added this to the 5.1.0 milestone Aug 1, 2016
@jbogard jbogard merged commit 842ad57 into LuckyPennySoftware:master Aug 1, 2016
@lbargaoanu lbargaoanu deleted the CustomObjectMapper branch August 2, 2016 08:51
@lock
Copy link

lock bot commented May 6, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators May 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants