Ticket #3691 (closed enhancement: fixed)
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
Change History
Changed 8 years ago by sergey
- Attachment 0001-Remove-.cl-file-extension-from-LISP-syntax.patch added
comment:3 Changed 8 years ago by zaytsev
- Cc zaytsev added
Andrew, shall we commit this for 4.8.18, looks safe enough to me?
comment:4 Changed 8 years 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 8 years 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 8 years 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
Applied as [97430c1fa4cd25501f22d31619afe1bc971bc922].
