RE: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-relatedfunctions.

2024-12-20 Thread Maxime Devos
> [...] > Maybe there's a misunderstanding here: What Adams test does is to compile a > list of results. What he refers to is the #t returned by the test checking > that the compiled results are correct. I changed that wording. Otherwise, I > think the test is fine (on a Unix system). Sounds re

Re: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-relatedfunctions.

2024-12-20 Thread Nala Ginrut
> > If so, the entire (ice-9 rdelim) module needs to be revised as well as > libguile/rdelim.c since the current implementation assumes that the line > delimiter is a single character, which was not true under Windows last time > I checked. > As I know, Windows seems the only mainstream and livin

Re: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-relatedfunctions.

2024-12-20 Thread Mikael Djurfeldt
Hi Maxime, On Fri, Dec 20, 2024 at 4:15 PM Maxime Devos wrote: > > Thanks for your patches! Applied now. > > > > Regardless of the question whether to generalise, there was also the thing > about the test testing for returning #true even though this behaviour isn’t > documented anywhere. > Mayb

RE: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-relatedfunctions.

2024-12-20 Thread Maxime Devos
> Thanks for your patches! Applied now. Regardless of the question whether to generalise, there was also the thing about the test testing for returning #true even though this behaviour isn’t documented anywhere. Also, there was the thing about needing to verify whether (ice-9 rdelim) always rec

Re: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-related functions.

2024-12-20 Thread Nala Ginrut
You typed so many patches unrelated thing that help me thought your time is cheap. It is cheap talking to me. Personally I'm so curious why you think blocklist is a weapon to threat people. But anyway, I don't against your interesting opinions, since you are always interesting. Anyway, I would li

RE: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-related functions.

2024-12-20 Thread Maxime Devos
>I don't talk nonsense with you, and I didn't against your opinions even for >your nonsense. >The only thing I care is that, are you ready to back to the coding part, and >show your code than cheap talking, now? This isn’t exactly what I wrote about in my last e-mail, but it’s a similar kind of

Re: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-related functions.

2024-12-20 Thread Nala Ginrut
I don't talk nonsense with you, and I didn't against your opinions even for your nonsense. The only thing I care is that, are you ready to back to the coding part, and show your code than cheap talking, now? Best regards. On Fri, Dec 20, 2024, 23:47 Maxime Devos wrote: > Can you stop it with th

RE: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-related functions.

2024-12-20 Thread Maxime Devos
Can you stop it with the repetitive claims of “this is just my proposal, I have freedom to state it, you don’t need to persuade me, etc.”? I’ve heard you the first, dunno, 5 or so times. Except for the “just” qualifier, I did not disagree anywhere. For why you shouldn’t do those repetitive claim

Re: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-related functions.

2024-12-20 Thread Nala Ginrut
"I also care if the image in your mind is not what showed in your mind" should be "showed in my mind" Best regards. On Fri, Dec 20, 2024, 23:30 Nala Ginrut wrote: > As I said, I didn't against any of your opinions. But I have my freedom to > comment on what I think important. > > So I made my p

Re: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-related functions.

2024-12-20 Thread Nala Ginrut
As I said, I didn't against any of your opinions. But I have my freedom to comment on what I think important. So I made my proposal accordingly to the specific issue as you pointed out. This may not be accepted by you, but that's your freedom to share your mind further. And I unnecessarily need to

RE: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-related functions.

2024-12-20 Thread Maxime Devos
>Here are the "back to the track" reply for folks in this thread. > >So the situation is more clear now. The newline in various OS need to >respectively tested. And my idea is to check OS via (uname) in test cases. >Now that it's in tests, I think we don't have to talk much about the >efficiency

Re: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-related functions.

2024-12-20 Thread Nala Ginrut
> >You can just say you've no idea about it, > > That’s true (i.e., I can say I have no idea about it), but I have no > reason to lie about it, so I won’t. I did have an idea, and I gave the idea > to you, as I mentioned in my two previous replies. > > Thanks! This sentence saves a lot of time. He

RE: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-related functions.

2024-12-20 Thread Maxime Devos
>>I already did. Also, no. Just because you know something is bad, doesn’t mean >>you know a better solution, so it doesn’t automatically follow that a better >>solution should be given. >You can just say you've no idea about it, That’s true (i.e., I can say I have no idea about it), but I have

Re: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-related functions.

2024-12-20 Thread Nala Ginrut
> > I already did. Also, no. Just because you know something is bad, doesn’t > mean you know a better solution, so it doesn’t automatically follow that a > better solution should be given. > You can just say you've no idea about it, such situation doesn't need to write so many words to explain it

RE: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-related functions.

2024-12-20 Thread Maxime Devos
>I have no any interest to persuade you, just show my opinion and suggestions. >And I also have no interest to argue with you about the design, because the >efforts has made according to you suggestions. This is a problem. Just making a proposal without following up on discussions of its merits

Re: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-related functions.

2024-12-20 Thread Nala Ginrut
I have no any interest to persuade you, just show my opinion and suggestions. And I also have no interest to argue with you about the design, because the efforts has made according to you suggestions. I'm trying to follow the idea to not waste any efforts have been made. My suggestion is to find

RE: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-related functions.

2024-12-20 Thread Maxime Devos
>>Also, you are assuming “\n” is a line delimiter. This is true under Unix >>according to the documentation. But it doesn’t say anything about non-Unix >>systems. >RnRs defined read-line to handle different newline properly. It’s named (ice-9 rdelim) not (rnrs rdelim). Perhaps (ice-9 rdelim) co

Re: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-related functions.

2024-12-20 Thread Nala Ginrut
> Also, you are assuming “\n” is a line delimiter. This is true under Unix > according to the documentation. But it doesn’t say anything about non-Unix > systems. > RnRs defined read-line to handle different newline properly. My original idea is to stick to a pure line string reader iterator helpe

RE: [PATCH] test-suite: Add tests for `for-rdelim-in-port`-related functions.

2024-12-20 Thread Maxime Devos
>I think these procedures are handy in common situations. There has been a >discussion about generalization. I have the feeling that such generalization >either already exists in some SRFI or that one should put some deep thinking >into how to represent flexible iteration in Scheme. If one sho