From: Atharva Raykar <raykar.ath@gmail.com>
To: git@vger.kernel.org
Cc: Atharva Raykar <raykar.ath@gmail.com>,
gitster@pobox.com, avarab@gmail.com, j6t@kdbg.org,
phillip.wood123@gmail.com
Subject: [GSoC][PATCH v2 0/1] userdiff: add support for scheme
Date: Sat, 3 Apr 2021 18:46:11 +0530 [thread overview]
Message-ID: <20210403131612.97194-1-raykar.ath@gmail.com> (raw)
In-Reply-To: <20210327173938.59391-1-raykar.ath@gmail.com>
Hello all,
This is v2 of the patch I sent to add userdiff support to Scheme. I have
modified my approach to be more inclusive of as many Schemes as possible and
thus added many more forms. Since Ævar suggested we veer on the side of
inclusion when talking about the Gerbil scheme syntax, I felt it made sense to
extend the driver to include common, but non-standard extensions of Scheme,
including the forms of Racket and Guile.
Forms added since last patch:
- Variants of define such as def, defsyntax etc as suggested by Phillip and the
other reviewers. - The library form of R6RS, as well as module definitions of
Racket, Gerbil and Guile. * These forms were added on the recommendation of
Göran Weinholt, creator of the Akku scheme package manager and the Loko
Scheme implementation:
https://groups.google.com/g/comp.lang.scheme/c/Aczn0TNEr5g/m/Jq3AlKvZBgAJ -
The Racket forms for defining structs and classes.
I have restricted the "def" forms to only certain keywords, so that it does not
over-match to words like "deflate", "deform", "defer" etc.
I have also allowed the use of "/" after a define or def form, as some scheme
code uses it as a convention for defines in a certain context, such as Racket's
"define/public".
I have also fixed some a test case which had a redundant "ChangeMe".
Finally in the word regex, which has been simplified a lot, while retaining the
same functioning after taking into account Junio's suggestions.
Atharva Raykar (1):
userdiff: add support for Scheme
Documentation/gitattributes.txt | 2 ++
t/t4018-diff-funcname.sh | 1 +
t/t4018/scheme-class | 7 +++++++
t/t4018/scheme-def | 4 ++++
t/t4018/scheme-def-variant | 4 ++++
t/t4018/scheme-define-slash-public | 7 +++++++
t/t4018/scheme-define-syntax | 8 ++++++++
t/t4018/scheme-define-variant | 4 ++++
t/t4018/scheme-library | 11 +++++++++++
t/t4018/scheme-local-define | 4 ++++
t/t4018/scheme-module | 6 ++++++
t/t4018/scheme-top-level-define | 4 ++++
t/t4018/scheme-user-defined-define | 6 ++++++
t/t4034-diff-words.sh | 1 +
t/t4034/scheme/expect | 10 ++++++++++
t/t4034/scheme/post | 5 +++++
t/t4034/scheme/pre | 5 +++++
userdiff.c | 4 ++++
18 files changed, 93 insertions(+)
create mode 100644 t/t4018/scheme-class
create mode 100644 t/t4018/scheme-def
create mode 100644 t/t4018/scheme-def-variant
create mode 100644 t/t4018/scheme-define-slash-public
create mode 100644 t/t4018/scheme-define-syntax
create mode 100644 t/t4018/scheme-define-variant
create mode 100644 t/t4018/scheme-library
create mode 100644 t/t4018/scheme-local-define
create mode 100644 t/t4018/scheme-module
create mode 100644 t/t4018/scheme-top-level-define
create mode 100644 t/t4018/scheme-user-defined-define
create mode 100644 t/t4034/scheme/expect
create mode 100644 t/t4034/scheme/post
create mode 100644 t/t4034/scheme/pre
--
2.31.1
next prev parent reply other threads:[~2021-04-03 13:17 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-27 17:39 [GSOC][PATCH] userdiff: add support for Scheme Atharva Raykar
2021-03-27 22:50 ` Junio C Hamano
2021-03-27 23:09 ` Junio C Hamano
2021-03-28 3:16 ` Ævar Arnfjörð Bjarmason
2021-03-28 5:37 ` Junio C Hamano
2021-03-28 12:40 ` Atharva Raykar
2021-03-29 10:08 ` Phillip Wood
2021-03-30 6:41 ` Atharva Raykar
2021-03-30 12:56 ` Ævar Arnfjörð Bjarmason
2021-03-30 13:48 ` Atharva Raykar
2021-03-28 12:45 ` Atharva Raykar
2021-03-28 11:51 ` Atharva Raykar
2021-03-28 18:06 ` Junio C Hamano
2021-03-29 8:12 ` Atharva Raykar
2021-03-29 20:47 ` Junio C Hamano
2021-03-29 10:12 ` Phillip Wood
2021-03-27 23:46 ` Johannes Sixt
2021-03-28 12:23 ` Atharva Raykar
2021-03-29 10:18 ` Phillip Wood
2021-03-29 10:48 ` Johannes Sixt
2021-03-29 13:12 ` Ævar Arnfjörð Bjarmason
2021-03-29 14:06 ` Phillip Wood
2021-03-30 7:04 ` Atharva Raykar
2021-03-30 10:22 ` Atharva Raykar
2021-04-05 10:04 ` Phillip Wood
2021-04-05 17:58 ` Johannes Sixt
2021-04-06 12:29 ` Atharva Raykar
2021-04-06 19:10 ` Phillip Wood
2021-04-03 13:16 ` Atharva Raykar [this message]
2021-04-03 13:16 ` [GSoC][PATCH v2 1/1] userdiff: add support for scheme Atharva Raykar
2021-04-05 10:21 ` Phillip Wood
2021-04-06 10:32 ` Atharva Raykar
2021-04-08 9:14 ` [GSoC][PATCH v3 0/1] " Atharva Raykar
2021-04-08 9:14 ` [GSoC][PATCH v3 1/1] userdiff: add support for Scheme Atharva Raykar
2021-04-12 23:04 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210403131612.97194-1-raykar.ath@gmail.com \
--to=raykar.ath@gmail.com \
--cc=avarab@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=j6t@kdbg.org \
--cc=phillip.wood123@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).