Changes between Initial Version and Version 2 of Ticket #7897


Ignore:
Timestamp:
01/12/10 02:45:40 (12 years ago)
Author:
novoselt
Comment:

I changed my mind about importance of "restart" command in the middle of the code after seeing a talk today. So the patch is rewritten to allow it everywhere. All doctests still pass with both patches applied.

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #7897

    • Property Status changed from new to needs_review

  • Ticket #7897 – Description

    initial v2  
    55 - empty/whitespace/comment lines can be executed;
    66 - multiline commands can be executed;
    7  - stripping output prompts will not cut error messages if they occur (e.g. the first call "help Thing" currently shows some errors in Macaulay2).
     7 - stripping output prompts will not cut error messages if they occur (e.g. the first call "help Thing" currently shows some errors in Macaulay2);
     8 - "macaulay2.restart()" does not print a useless variable name.
    89
    910This is done by:
    … …  
    1213 - making sure that only output labels and spaces are stripped from the output.
    1314
    14 Side effects / Remaining issues:
    15  - "restart" command of Macaulay2 is handled separately when it is called like "macaulay2.restart()" since we need to repeat prompt adjustments;
    16  - this command cannot be used in the string code passed to Macaulay2, since it will cause a lock. Since this should not cause loss of data (if the user intentionally tried to restart Macaulay2), I think this is OK. Correct checking of all the code for "restart" in it would involve also checking if it is inside string constants.
    17 
    1815These patches make tickets #7882 and #7888 unnecessary.