Hi Juliana,

Juliana Sims <j...@incana.org> writes:

> Hi y'all,
> Thanks for the (continued) reviews!
>
>> I hadn't commented on that last sentence before, but if I knew how to
>> have the Guile debugger reliably break where I want it to (I don't, or
>> somehow haven't managed to have it work well), I don't think using
>> 'pk',
>> which requires editing files before and after debugging, could be
>> described as more convenient :-).
>
> A fair point!  I can change that wording in a next version of the patch.
>
>> I would suggest to apply the ’pk’ on the other branch, something as:
>> --8<---------------cut here---------------start------------->8---
>> (map (lambda (v)
>>        (if (number? v)
>>            (pk 'number v (number->string v))
>>            v))
>>      '(1 "2" "3" 4))
>> --8<---------------cut here---------------end--------------->8---
>
> I'm not sure I understand how this improves the demonstration of 'pk'.
> What does this form of the example demonstrate that the version in the
> patch does not?  It's a minor change so I'm happy to make it; I just
> want to ensure that we have the best possible version of the solution
> to the problem you see.

I'm not sure I understand the motivation too. To me the v2 patch looked
fine as-is.

-- 
Thanks,
Maxim



  • bug#71684: [PAT... Bug reports for GUILE, GNU's Ubiquitous Extension Language
    • bug#71684:... Maxim Cournoyer
      • bug#71... Bug reports for GUILE, GNU's Ubiquitous Extension Language
        • bu... Maxim Cournoyer
          • ... Maxim Cournoyer
        • bu... Simon Tournier
          • ... Maxim Cournoyer
            • ... Simon Tournier
              • ... Bug reports for GUILE, GNU's Ubiquitous Extension Language
                • ... Bug reports for GUILE, GNU's Ubiquitous Extension Language
                • ... Maxim Cournoyer
                • ... Simon Tournier
                • ... Bug reports for GUILE, GNU's Ubiquitous Extension Language
                • ... Janneke Nieuwenhuizen
                • ... Bug reports for GUILE, GNU's Ubiquitous Extension Language
                • ... Ludovic Courtès

Reply via email to