astmatcher-demo exposes major bugs in bclasp 0.3-test-7-8-g42f5e53

[Georgiy Tugai]

Most of these issues have been resolved as of 0.3-test-8-4-g653ee86,
however the "Bad keyword argument" error is still extant.

Regards,
Georgiy

On 09 Jul, Georgiy Tugai wrote:
> I decided to test astmatcher-demo[1] (known to be working on f7948ae
> i.e. tip of master) on bclasp 0.3-test-7-8-g42f5e53 (tip of newbuild at
> the time).
> 
> First, there is a missing external symbol
> NEW-DEDUPLICATING-REFACTORING-TOOL (called from
> src/lisp/clang-tool.lsp). I assumed that NEW-REFACTORING-TOOL would be a
> suitable replacement, evaluated the calling function definition (C-M-x
> in SLIME) and moved on.
> 
> Then, a series of undefined symbols (see Table 1) followed, each one
> "fixed" by C-M-x.
> 
> After this, astmatcher-demo's GET-CLASSES fails with "Bad keyword
> argument" followed by a large number (for example, 69993087673325; a
> decimal pointer address?) which was different between bclasp instances.
> 
> On further investigation, it is interesting to note that LOADing
> clang-tool.lsp rather than REQUIREing 'clang-tool results in more
> undefined symbol errors (including the external symbol
> REPLACEMENTS-AS-LIST). I believe this is indicative of a significant
> difference between evaluation and compilation in bclasp.
> 
> Table 1: clang-tool.lsp symbols misreported as undefined
> +----------------+--------------------------------------+
> |           DEFUN|IDENTIFY-HINT-ENVIRONMENT             |
> |DEFINE-CONDITION|NODE-MATCHER-AMBIGUOUS-ERROR          |
> |           DEFUN|ERROR-UNLESS-VALID-PREDICATE          |
> |     DEFCONSTANT|+NODE-MATCHERS+                       |
> |           DEFUN|IDENTIFY-NODE-MATCHER-ENVIRONMENT     |
> |     DEFCONSTANT|+TRAVERSAL-MATCHERS+                  |
> |           DEFUN|IDENTIFY-TRAVERSAL-MATCHER-ENVIRONMENT|
> |           DEFUN|APPLICABLE-MATCHER-P                  |
> |           DEFUN|MATCH-RUN                             |
> +----------------+--------------------------------------+
> 
> References
> [1] https://github.com/drmeister/astmatcher-demo


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <https://mailman.common-lisp.net/pipermail/clasp-devel/attachments/20150710/6f31bf70/attachment.sig>