fcl-md: cosmetic
all files:
- better use word 'Markdown' is this casing: 'Markdown', not 'MarkDown'. as you see on the site https://www.markdownguide.org/ this casing is the canonical. currently you have class names like TMarkDownTextBlock, TMarkDownLeafBlock, TMarkDownDocument, TMarkDownElement etc.
markdown.elements.pas:
-
ids
TWhitespaceMode,TPosition,TNodeStyle,TTextNodeKindneed the prefix (e.g. TMarkdown) to not pollute the autocompletion and avoid future conflicts, half of them are generic names. -
better add prefix to this const:
GrowSize = 10;. or it can be moved intoimplemenationpart. this id is not used out the unit.
markdown.fpdocrender.pas:
- ids
TElementType,TSectionType,SectionNodeNames,ElementNodeNames- too generic, maybe add prefix?
markdown.utils.pas:
-
THashTableand maybeTUnicodeCharDynArrayand many functions here, e.g.IsWhitespaceCharIsWhitespaceMustEscapeetc - too generic, maybe add prefix? or move into some container class (asstaticfuncs to keep speed)?
Edited by Alexey Torgashin