GoogleCodeExporter
commented
9 years ago I wanted this too and created a patch for it. The problems I saw were:
1. Constants used for options => No way to use this as a library without
compiling
from source.
2. Needed to be able to set options from configuration file.
So what this patch does is the following:
1. New class, MarkdownOptions, which contains all the options that were
previously
constants. The class is immutable, the options can be set explicitly at
construction
time, or it can use the defaults that were used before, or use the defaults +
overrides from configuration files. The config options are placed in
appSettings and
take the form Markdown.PropertyName, e.g. Markdown.AutoHyperlink. (Maybe later
this
should use its own config section but that seemed overkill for now).
Having the options per instance creates some other changes as well. The regular
expressions that use the options are now per instance, not static anymore. The
ones
that don't use any options are still static.
2. Realistically for most cases you only need a single instance, and probably
most of
the time you'd want defaults + config overrides. So there is a static property,
Default, on the Markdown class so you can just go Markdown.Default.Transform(s)
if
that's all you want.
3. The Markdown class is immutable so if you want to change options later you
must
create a new instance. This avoids having to keep track of option changes to
recompile regular expressions. Immutable == good :)
And that's about it. Added bin/obj to the ignore list as well.
I'll probably look at the failing unit tests later this week if I have a chance.
Original comment by hviturha...@gmail.com on 4 Jan 2010 at 11:26
Attachments:
Original issue reported on code.google.com by
kevin.ba...@gmail.comon 4 Jan 2010 at 2:41