Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement IFormattable on Option<T> #21

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

Commits on Dec 23, 2016

  1. Return string value of Some or blank in Option<T>.ToString()

    This renders Option<T>.ToString() consistent with Nullable<T>.ToString() and easier to substitute for.
    atifaziz committed Dec 23, 2016
    Configuration menu
    Copy the full SHA
    c9b9959 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    f4668ce View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    f753f1a View commit details
    Browse the repository at this point in the history

Commits on Dec 24, 2016

  1. Simpler & less redundant IFormattable implementation

    Instead of hopping over another String.Format (and causing an
    allocation of another format string), delegate to the underlying
    implementation of IFormattable on the value if it exists, otherwise to
    Option's own ToString().
    atifaziz committed Dec 24, 2016
    Configuration menu
    Copy the full SHA
    c2f0318 View commit details
    Browse the repository at this point in the history

Commits on Aug 23, 2017

  1. Configuration menu
    Copy the full SHA
    7c5b448 View commit details
    Browse the repository at this point in the history

Commits on Jul 22, 2019

  1. Configuration menu
    Copy the full SHA
    e0f0c90 View commit details
    Browse the repository at this point in the history