Fix keyword typos, support bare raise and property re-publication

[wpostma]

Summary

Three grammar fixes for Delphi/Object Pascal constructs that produce parse errors on valid code:

• Fix kSafecall regex typo: /safecal/i → /safecall/i — the missing trailing l meant safecall was never recognized as a keyword, causing thousands of errors in COM type library imports (MSHTML, Word_TLB, mscorlib_TLB, etc.)
• Fix kSealed regex typo: /seled/i → /sealed/i — sealed classes were never recognized
• Support bare raise;: Delphi allows raise; without an expression to re-raise the current exception inside an except block. Made the expression field optional.
• Support property re-publication: Delphi descendant classes can re-publish inherited properties without repeating the type declaration, e.g. property Align;, property Font;, optionally with specifiers like property Visible stored False; or property BookmarkSize write SetBookmarkSize;. Made the : type portion of declProp optional.

Testing

Validated against 400 unique .pas files (182.6 MB, 5.1M lines) from real-world Delphi codebases including Embarcadero RTL/FMX/VCL sources, TMS components, ElevateDB, JCL/JEDI, mORMot, DevExpress, and business applications.

Results from specific files:

File	Errors before	Errors after	Notes
FMX.StdCtrls.pas	1,598	0	100% clean
FireDAC.Comp.DataSet.pas	2,118	0	100% clean
edblocal.pas	474	14	bare raise fix
AdvToolBar.pas	612	312	remaining errors are other gaps
MSHTML.pas	11,706	6,087	safecall fix; remaining are readonly/dispinterface gaps

Grammar changes

 # kSafecall typo
-kSafecall: $ => /safecal/i,
+kSafecall: $ => /safecall/i,

 # kSealed typo
-kSealed: $ => /seled/i,
+kSealed: $ => /sealed/i,

 # Bare raise
-field('exception', $._expr),
+field('exception', optional($._expr)),

 # Property re-publication
-':',
-field('type', $.type),
+optional(seq(':', field('type', $.type))),

# JNI Bindings  allow $ in property names, such as read _Getval$transformer
  [JavaSignature('com/clover/core/internal/Lists$1')]
  JLists_1 = interface(JIterable)
    ['{7BC2900F-BE04-4ACD-B54B-EF426100E8A9}']
    function _Getval$transformer: JLists_Transformer; cdecl;
    property val$transformer: JLists_Transformer read _Getval$transformer;
  end;

[wpostma]

Fixes top three issues in
#19

PR'ing them because they're the lowest risk changes, and I have validated against 400 of my largest pas files in my local working folders.

[wpostma]

Note on the $ identifier fix (commit b34c593): The $ character in identifiers is not standard Object Pascal — it comes specifically from Delphi's JNI/Android binding generator, which translates Java nested class names like Address.CacheKey into Address$CacheKey. This is the only context where $ appears in real-world Delphi identifiers, confirmed by scanning 400 large .pas files from diverse codebases.

[AntoineGS]

I've run into the bare raise issue in a current use of this, I would be happy to see this merged :)

[wpostma]

I've run into the bare raise issue in a current use of this, I would be happy to see this merged :)

Can you pull this branch in from my fork and try it with your cases?

Testing effort would probably get it merged.

[AntoineGS]

I can whip something together, I was using the crate as-is but I can use a local repo instead.
My use case is still in its early stages so testing will not be as useful as a more mature project :)

[wpostma]

@Isopod any interest in this pr ?

[AntoineGS]

I can confirm it resolves my issue with the bare raise statement!