~zge

Germany, Bavaria

https://ruzkuku.com/

CS student. I use and program {Lisp,C,Go,*nix} systems.

~zge/public-inbox

Last active 23 days ago
View more

Recent activity

Index out of Range a month ago

From Philip K. to ~emersion/soju-dev

Hi, I've just updated soju and got this error message after
(accidentally) running /list on IRCNet:

panic: runtime error: index out of range [15] with length 15

goroutine 5 [running]:
git.sr.ht/~emersion/soju.partialCasemap(0x96d3b0, 0xc0003aa046, 0xf, 0xc0002d2c00, 0x414148) 
        /home/phi/soju/irc.go:505 +0x247
git.sr.ht/~emersion/soju.(*downstreamConn).marshalEntity(0xc000092a20, 0xc000206140, 0xc0003aa046, 0xf, 0xc0002d2d30, 0x4138d6)
        /home/phi/soju/downstream.go:214 +0x77
git.sr.ht/~emersion/soju.(*upstreamConn).handleMessage.func11(0xc000092a20)                  
        /home/phi/soju/upstream.go:1084 +0xc5
git.sr.ht/~emersion/soju.(*upstreamConn).forEachDownstreamByID.func1(0xc000092a20)

Re: mu4e failures with last changes a month ago

From Philip Kaludercic to ~zge/public-inbox

Thierry Volpiatto <thievol@posteo.net> writes:

> Philip Kaludercic <philipk@posteo.net> writes:
>
>> Thierry Volpiatto <thievol@posteo.net> writes:
>>
>>> You should declare autocrypt-loaded-version with a default value
>>> otherwise we have an error void variable...etc which break even the
>>> mua
>>> in use (mu4e for me).
>>
>> The issue was that I forgot to quote autocrypt-loaded-version when
>> checking if it was bound.
>

Re: mu4e failures with last changes a month ago

From Philip Kaludercic to ~zge/public-inbox

Thierry Volpiatto <thievol@posteo.net> writes:

> You should declare autocrypt-loaded-version with a default value
> otherwise we have an error void variable...etc which break even the mua
> in use (mu4e for me).

The issue was that I forgot to quote autocrypt-loaded-version when
checking if it was bound.

> Once this is fixed, it is possible to read mails with mu4e, however
> sending is broken (I send here with an autocrypt reverted to 4824fb6) by
> generic functions.
>
> no applicable method autocrypt-get-header ...

Re: [PATCH 3/3] When inserting into the list, don't allow duplicate emails 4 months ago

From Philip K. to ~zge/public-inbox

Thierry Volpiatto <thievol@posteo.net> writes:

> Samuel W. Flint <swflint@flintfam.org> writes:
>
>> +  (pushnew (list email fingerprint preference) autocrypt-accounts
>> +           :key #'first :test #'string=)
>
> Use `cl-pushnew` instead part of cl-lib.el.

And setting that aside, (cl-)pushnew will silently "fail" if in this
case the alist already contains `email'. I think it would be a better
idea to do something like

Re: [PATCH v2] Include documentation for all hooks 4 months ago

From Philip K. to ~zge/public-inbox

"Samuel W. Flint" <swflint@flintfam.org> writes:

> ---
>  README.md | 23 ++++++++++++++++-------
>  1 file changed, 16 insertions(+), 7 deletions(-)
>
> diff --git a/README.md b/README.md
> index 213b57a..0cb2b27 100644
> --- a/README.md
> +++ b/README.md
> @@ -26,13 +26,8 @@ integration is welcomed.
>  How to use
>  ----------
>  

Re: [PATCH 2/3] Move autocrypt-accounts insertions to dedicated, interactive function 4 months ago

From Philip K. to ~zge/public-inbox

"Samuel W. Flint" <swflint@flintfam.org> writes:

> This allows the use of pre-existing keys, and makes sure that saving
> of this information is done the same way, at all times.
> ---
>  autocrypt.el | 19 +++++++++++++++++--
>  1 file changed, 17 insertions(+), 2 deletions(-)
>
> diff --git a/autocrypt.el b/autocrypt.el
> index 5fd6b6f..a3f215f 100644
> --- a/autocrypt.el
> +++ b/autocrypt.el
> @@ -510,11 +510,26 @@ Will handle and remove \"Do-(Discourage-)Autocrypt\" if found."
>      (let ((res (epg-context-result-for ctx 'generate-key)))

Re: [PATCH 1/3] Save autocrypt accounts data uniformly 4 months ago

From Philip K. to ~zge/public-inbox

"Samuel W. Flint" <swflint@flintfam.org> writes:

> ---
>  autocrypt.el | 7 ++-----
>  1 file changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/autocrypt.el b/autocrypt.el
> index 965f661..5fd6b6f 100644
> --- a/autocrypt.el
> +++ b/autocrypt.el
> @@ -510,11 +510,8 @@ Will handle and remove \"Do-(Discourage-)Autocrypt\" if found."
>      (let ((res (epg-context-result-for ctx 'generate-key)))
>        (unless res
>          (error "Could not determine fingerprint"))

Re: [PATCH] Updating README to document integration with all supported 4 months ago

From Philip K. to ~zge/public-inbox

"Samuel W. Flint" <swflint@flintfam.org> writes:

> Hello,
>
> In your README, you present a configuration for using rmail, and
> leaves other users to dig into code do discover what hooks to add
> autocrypt-mode to.  To correct this, I've added a small bit to the
> README that can (hopefully) help clear it up.

Honestly, looking at the README, I think it's better to just rewrite the
entire "how to use section". I have stopped using use-package, and have
since realized that this convenience for some might not be that
appreciated by others. And in general, it would be nice to have a
minimal configuration for all supported mail systems.

[PATCH] Ensure port number is valid 5 months ago

From Philip K to ~martijnbraam/public-inbox

---
 main.c | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/main.c b/main.c
index eed5e7d..4528da5 100644
--- a/main.c
+++ b/main.c
@@ -1414,8 +1414,15 @@ main(int argc, char *argv[])
	int opt;
	while ((opt = getopt(argc, argv, "hp:")) != -1) {
		switch (opt) {
			case 'p':
				port = strtol(optarg, &endptr, 10);
[message trimmed]

Re: [PATCH 5/5] Generalize broadcast 5 months ago

From Philip K. to ~martijnbraam/public-inbox

"Eyal Sawady" <ecs@d2evs.net> writes:

> [1:text/plain Hide]
>
> I don't like the macro trickery here, I'd rather do something like
>
> enum broadcast_mode {
> 	MODE_ALL = 1 << 0,
> 	MODE_ONLY = 1 << 1,
> 	MODE_NOT = 1 << 2,
> 	MODE_GHOST = 1 << 3,
> };
>
> and pass a player index into broadcast()