Skip to content
This repository was archived by the owner on May 16, 2022. It is now read-only.

Add explicit formatter declaration for ZeroFormattable attribute #81

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Anapher
Copy link

@Anapher Anapher commented Aug 13, 2017

  • add tests
  • add checks in analyzer

I find it rather complicated having to register the formatters one after one. First, advanced logic is needed to register all resolvers from all referenced libraries (were should that code be put?) and it's uneconomical to keep track of all needed resolvers in a huge repository. I implemented a behavior like Json.Net and XmlSerializer are using: Every type knows it's formatter which is just used instead of resolving one (if it's set). The registering still has it's use when dealing with external types, but for custom types the FormatterType is a huge advantage.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant