Re: multiple values: value1 treated extra even when not in register
Daniel Jour <[email protected]>
| Newsgroups | gmane.lisp.clisp.devel |
|---|---|
| Message-ID | <CACZHyTNUFr6hhiEQB1Esb1xAgTofh-hYh-YBjOkvhYcpX3tr9Q@mail.gmail.com> |
Sam wrote: > There will be so many such minor things that you will be able to spend > the whole summer on them. Jörg wrote: > I disagree with the suggested treatment. Daniel found a bug, all by > himself. The best reward is to let him proudly fix it immediately, so > he can forget about this particular task and lessen the burden on his > brain. I believe the direct reward to be more motivating than > "maybe in a couple of months". Thank you, though it's not really about the "reward" but more about understanding "why not": It's a bug that, by itself, causes no harm. It could have lead to serious headaches when hunting a possible other bug in the multiple value code, because one does not expect value1 to be treated any different when it's not in a register. Thus, it could have happended that I'd looked at the wrong code paths (the one without extra treatment). Regarding the time: It's already spent, and it didn't cost much "extra" because what I was actually doing when I found it was reading the code in order to understand how all (or better: the core parts) of it works. Sam wrote: > He should create a bug report or a patch and leave it in the SF bug > tracker for later. That's what I'll do for now, then. (The change is already applied in the forked hg repo, though. I guess reverting it to later bring it back in might be a bit odd.) https://sourceforge.net/p/clisp/patches/45/ ------------------------------------------------------------------------------ Mobile security can be enabling, not merely restricting. Employees who bring their own devices (BYOD) to work are irked by the imposition of MDM restrictions. Mobile Device Manager Plus allows you to control only the apps on BYO-devices by containerizing them, leaving personal data untouched! https://ad.doubleclick.net/ddm/clk/304595813;131938128;j _______________________________________________ clisp-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/clisp-devel