Ticket #3691 (closed enhancement: fixed)

Opened 6 months ago

Last modified 4 months ago

Remove .cl file extension from LISP programs

Reported by: sergey Owned by:
Priority: major Milestone: 4.8.19
Component: mcedit Version: master
Keywords: Cc: zaytsev
Blocked By: Blocking: #3690
Branch state: no branch Votes for changeset: committed-master

Description

As was discussed a bit in ticket #3690 this file extension is no longer used for LISP programs. Removing it from LISP will allow us to add OpenCL syntax to mcedit.

Doing it as a separate patch to keep changes a bit more self-contained and to avoid one commit to one syntax touching other syntax files.

Attachments

0001-Remove-.cl-file-extension-from-LISP-syntax.patch (1004 bytes) - added by sergey 6 months ago.

Change History

comment:1 Changed 6 months ago by andrew_b

  • Blocking 3690 added

comment:2 Changed 6 months ago by andrew_b

  • Component changed from mc-core to mcedit

comment:3 Changed 6 months ago by zaytsev

  • Cc zaytsev added

Andrew, shall we commit this for 4.8.18, looks safe enough to me?

comment:4 Changed 6 months ago by zaytsev

Ah, I see, it only makes sense together with the other ticket. Ok, let's do it after release then, who wants to (potentially) upset Lisp programmers for no good reason...

comment:5 Changed 6 months ago by sergey

Yep, it's only crucial for #3690. Just kinda preferred to separate commits and discussion between syntax itself and change which could be treated as a regression by LISP programmers.

If the release is planned soon, then indeed better to wait for until after the release, so if the change actually upsets someone it'll be more time to rewind the change or reconsider it.

Just let me know if some further tweaks are required for either of the change, i'll be happy to address them :)

comment:6 Changed 4 months ago by andrew_b

  • Status changed from new to closed
  • Votes for changeset set to committed-master
  • Resolution set to fixed
  • Milestone changed from Future Releases to 4.8.19
Note: See TracTickets for help on using tickets.