flexi-streams icon indicating copy to clipboard operation
flexi-streams copied to clipboard

Fix symbol internalization in KEYWORD package

Open kilianmh opened this issue 3 years ago • 4 comments

Replace keywords in defpackage :export and :shadow clauses with uninterned symbols. This avoids interning them in *package*, which should simplify autocompletion and may improve garbage collection.

kilianmh avatar Feb 23 '22 21:02 kilianmh

Can you get the symbol names in the import-from clause in the test package?

gefjon avatar Feb 23 '22 23:02 gefjon

The symbol names in the test package :import-from clause are now also uninterned instead of keywords.

kilianmh avatar Feb 24 '22 09:02 kilianmh

Reverting test package :IMPORT-FROM clause package name to KEYWORD, which should improve readability without unnecessary expansion of *PACKAGE*.

kilianmh avatar Feb 24 '22 11:02 kilianmh

Summary:

Fix symbol internalization in KEYWORD package

  • Replace KEYWORD (except library-names) with STRING in FLEXI-STREAMS.ASD
  • Change DEFPACKAGE shadowed, imported, exported symbols to uninterned symbols

-> Reduction of KEYWORD package symbols by 54 when loading FLEXI-STREAMS -> Improved KEYWORD auto-correction

kilianmh avatar Jul 22 '22 19:07 kilianmh