~selene

Phoenix, AZ

https://rsaihe.dev

Recent activity

Re: [PATCH hare.vim 00/11] Syntax reorganization 5 days ago

From Amelia Clarke to ~sircmpwn/hare-dev

On Thu Sep 22, 2022 at 2:03 AM MST, Drew DeVault wrote:
> Okay, no worries. Can you send a patch for hare.vim and then prepare
> something to go upstream?

From what Bram said, it sounds like the best way to get it upstream is
by opening a pull request, so I will go ahead and get started on that.

[PATCH hare.vim] Update headers to conform to style used by Vim 6 days ago

From Amelia Clarke to ~sircmpwn/hare-dev

Signed-off-by: Amelia Clarke <me@rsaihe.dev>
---
 compiler/hare.vim | 2 ++
 ftplugin/hare.vim | 8 +++++---
 indent/hare.vim   | 2 ++
 syntax/hare.vim   | 2 ++
 4 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/compiler/hare.vim b/compiler/hare.vim
index adc4710..c0fa68c 100644
--- a/compiler/hare.vim
+++ b/compiler/hare.vim
@@ -1,5 +1,7 @@
" Vim compiler file
[message trimmed]

Re: [PATCH hare.vim 00/11] Syntax reorganization 10 days ago

From Amelia Clarke to ~sircmpwn/hare-dev

Alright, I just received a response from Bram.

Apparently the filetype is already being detected, but for everything
else we will need to ensure that the header for each file includes
contact info and a "last change" date in order to conform to the style
used in the Vim repository.

Re: Switch style 30 days ago

From Amelia Clarke to ~sircmpwn/hare-users

On Sun Aug 28, 2022 at 6:34 PM MST, Byron Torres wrote:
> switch (o) {
> case
>         opcode::EQ, opcode::LT, opcode::GT, opcode::ADD,
>         opcode::SUB, opcode::MUL, opcode::DIVMOD
> =>
>         calc(dat, o)?;
> case =>
>         void;
> };

Oh, that's kind of nice looking. I might consider doing something like
that as an alternative.

Switch style 30 days ago

From Amelia Clarke to ~sircmpwn/hare-users

I was wondering if there is an existing convention for how to properly
handle long lines in switch expressions. There doesn't seem to be an
example of this in the style guide, though I could also see convention
A3 applying to this situation.

An example from the personal project I am currently working on:

	switch (o) {
	case opcode::EQ, opcode::LT, opcode::GT, opcode::ADD,
			opcode::SUB, opcode::MUL, opcode::DIVMOD =>
		calc(dat, o)?;
	case => void;
	};

Re: [PATCH hare.vim 00/11] Syntax reorganization a month ago

From Amelia Clarke to ~sircmpwn/hare-dev

Alright, I can start work on that tomorrow.

Re: [PATCH hare.vim 00/11] Syntax reorganization a month ago

From Amelia Clarke to ~sircmpwn/hare-dev

I would definitely be interested!

I have been working a bit on a few other miscellaneous improvements,
especially for the error assertion operator, but they aren't completely
ready yet. Do you think it would be worthwhile to wait for those before
properly sending the plugin upstream?

In the meantime I can get in touch with Bram and begin the discussion.

Re: Casting to enums a month ago

From Amelia Clarke to ~sircmpwn/hare-users

On Tue Aug 23, 2022 at 5:17 PM MST, Bor Grošelj Simić wrote:
> This was already proposed, but got rejected because it's pretty hard
> to define and implement checks for what counts as "valid" in a certain
> enum.

Yeah, that makes sense.

[PATCH hare.vim 11/11] syntax: improve use statement a month ago

From Amelia Clarke to ~sircmpwn/hare-dev

Use statements are now highlighted correctly in all cases, including
when there is leading whitespace or when they take up multiple lines,
including comments inside.

Signed-off-by: Amelia Clarke <me@rsaihe.dev>
---
 syntax/hare.vim | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/syntax/hare.vim b/syntax/hare.vim
index 55a7215..65f8b46 100644
--- a/syntax/hare.vim
+++ b/syntax/hare.vim
@@ -90,7 +90,9 @@ syn match hareType "\v<size>((\_s*//.*\_$)*\_s*\()@!" contains=hareComment
[message trimmed]

[PATCH hare.vim 10/11] syntax: remove '..' and '...' operators a month ago

From Amelia Clarke to ~sircmpwn/hare-dev

Signed-off-by: Amelia Clarke <me@rsaihe.dev>
---
This is likely just personal preference, but I find having '..' and
'...' highlighted is somewhat distracting.

 syntax/hare.vim | 1 -
 1 file changed, 1 deletion(-)

diff --git a/syntax/hare.vim b/syntax/hare.vim
index df4dd47..55a7215 100644
--- a/syntax/hare.vim
+++ b/syntax/hare.vim
@@ -91,7 +91,6 @@ syn match hareSpaceError "\v\s+$" display excludenl
syn match hareSpaceError "\v\zs +\ze\t" display
[message trimmed]