Re: [PATCH] web: allow uri-encode to uppercase the percent-encoding

2013-05-02 Thread Daniel Hartwig
On 2 May 2013 23:27, Aleix Conchillo Flaqué wrote: > This patch allows uri-encode to uppercase the percent-encoded part. So, > > (uri-encode "/") > => %2f > > (uri-encode "/" #:uppercase #t) > => %2F Hi Why make this optional, it seems like unnecessary fluff? Both cases are equivalent, and RFC

Re: [PATCH] web: allow uri-encode to uppercase the percent-encoding

2013-05-02 Thread Daniel Hartwig
On 3 May 2013 09:00, Daniel Hartwig wrote: > On 2 May 2013 23:27, Aleix Conchillo Flaqué wrote: >> This patch allows uri-encode to uppercase the percent-encoded part. So, >> >> (uri-encode "/") >> => %2f >> >> (uri-encode "/" #:uppercase #t) >> => %2F > > Hi > > Why make this optional, it seems l

Re: [PATCH] web: uri-encode hexadecimal percent-encoding is now uppercase

2013-05-02 Thread Aleix Conchillo Flaqué
On Thu, May 2, 2013 at 2:17 PM, Mark H Weaver wrote: > Hi Aleix, > > There was one remaining problem with your commit: web-uri.test contained > some tests that assumed lowercase percent encoding. I took the liberty > of modifying your commit to fix those tests, and then pushed it to > stable-2.0.

Re: [PATCH] web: uri-encode hexadecimal percent-encoding is now uppercase

2013-05-02 Thread Mark H Weaver
Hi Aleix, There was one remaining problem with your commit: web-uri.test contained some tests that assumed lowercase percent encoding. I took the liberty of modifying your commit to fix those tests, and then pushed it to stable-2.0. Thanks! Mark

Re: [PATCH] web: uri-encode hexadecimal percent-encoding is now uppercase

2013-05-02 Thread Aleix Conchillo Flaqué
On Thu, May 2, 2013 at 11:53 AM, Mark H Weaver wrote: > Hi Aleix, > > This patch looks good to me, except that I don't like the variable name > 'num' for a string. Actually, I'm not sure we need to bind that to a > variable at all. How about just wrapping 'string-upcase' around the > call to 'nu

Re: [PATCH] web: uri-encode hexadecimal percent-encoding is now uppercase

2013-05-02 Thread Mark H Weaver
Hi Aleix, This patch looks good to me, except that I don't like the variable name 'num' for a string. Actually, I'm not sure we need to bind that to a variable at all. How about just wrapping 'string-upcase' around the call to 'number->string'? Thanks, Mark > @@ -396,11 +396,12

[PATCH] web: uri-encode hexadecimal percent-encoding is now uppercase

2013-05-02 Thread Aleix Conchillo Flaqué
This patch changes the way uri-encode does percent-encoding. The hexadecimal characters are now uppercase as suggested by RFC3986. So, (uri-encode "/") => %2F Aleix uri-encode-uppercase.patch Description: Binary data

Re: [ANN] An enhanced REPL for Guilers

2013-05-02 Thread Stjepan Horvat
Hi..i installed it on archlinux and when i write into guile prompt i get: guile> (use-modules (nala shell)) Backtrace: In standard input: 1: 0* (use-modules (nala shell)) 1: 1 (eval-case (# # *unspecified*) (else #)) 1: 2 (begin (process-use-modules (list (list #))) *unspecified*) In u

Re: [ANN] An enhanced REPL for Guilers

2013-05-02 Thread Stjepan Horvat
I didn't use su aldow i am using a root user as default..so i also start guile using root user.. On Thu, May 2, 2013 at 9:48 AM, Nala Ginrut wrote: > On Thu, 2013-05-02 at 08:44 +0200, Stjepan Horvat wrote: > > Hi..i installed it on archlinux and when i write into guile prompt i > > get: > > > g

[PATCH] web: allow uri-encode to uppercase the percent-encoding

2013-05-02 Thread Aleix Conchillo Flaqué
This patch allows uri-encode to uppercase the percent-encoded part. So, (uri-encode "/") => %2f (uri-encode "/" #:uppercase #t) => %2F Aleix uri-encode-uppercase.patch Description: Binary data

Re: Redo Safe Variables, New take

2013-05-02 Thread Stefan Israelsson Tampe
Hi William and thanks for your mail! On Thu, May 2, 2013 at 11:21 AM, William ML Leslie < william.leslie@gmail.com> wrote: > On 30 April 2013 06:15, Stefan Israelsson Tampe > wrote: > > Hi All, > > > > As I told you in an earlier mail I'm back cleaning up and reworking > > guile-log and > >

Re: wip-rtl disassembler

2013-05-02 Thread Noah Lavine
That's great! Thanks a lot for doing so much RTL stuff. Noah On Thu, May 2, 2013 at 4:11 AM, Andy Wingo wrote: > A brief note to say that (system vm disassembler) in wip-rtl now has a > functioning "disassemble-program" that finds function bounds using the > ELF symbol table. It's not great y

Re: attempt at array cleanup

2013-05-02 Thread Daniel Llorens
On May 1, 2013, at 01:10, Daniel Llorens wrote: > In the ra0 branch, that still fails, and this also fails: > > scheme@(guile-user)> #1b@1(#t #t) > While compiling expression: > ERROR: In procedure uniform-array->bytevector: Wrong type (expecting uniform > contiguous array): #1b@1(#t #t) I hav

Re: [ANN] An enhanced REPL for Guilers

2013-05-02 Thread Nala Ginrut
On Thu, 2013-05-02 at 11:31 +0200, Stjepan Horvat wrote: > I didn't use su aldow i am using a root user as default..so i also > start guile using root user.. > Could you checkout if the path `guile -c (display (car %load-path))' have nala directory? > On Thu, May 2, 2013 at 9:48 AM, Nala Ginrut

Re: [Feature Request] Some ideas on 'mmap'

2013-05-02 Thread Nala Ginrut
Here's the implementation in Artanis, and all the constant like PROT_READ definition are ignored: -- code - (define *libc-ffi* (dynamic-link)) (define %mmap (pointer->procedure '* (dynamic-func "mmap" *libc-ffi*) (list

Re: Redo Safe Variables, New take

2013-05-02 Thread William ML Leslie
On 30 April 2013 06:15, Stefan Israelsson Tampe wrote: > Hi All, > > As I told you in an earlier mail I'm back cleaning up and reworking > guile-log and > refreshing the memory of the inner details of that code base enabled me to > rewrite > the spec for redo safe variables considerable. I think t

wip-rtl disassembler

2013-05-02 Thread Andy Wingo
A brief note to say that (system vm disassembler) in wip-rtl now has a functioning "disassemble-program" that finds function bounds using the ELF symbol table. It's not great yet; would like to work on it a bit more this evening, but it's better than what we had ;) Andy -- http://wingolog.org/

Re: [ANN] An enhanced REPL for Guilers

2013-05-02 Thread Nala Ginrut
On Thu, 2013-05-02 at 08:44 +0200, Stjepan Horvat wrote: > Hi..i installed it on archlinux and when i write into guile prompt i > get: > guile> (use-modules (nala shell)) > Backtrace: > In standard input: >1: 0* (use-modules (nala shell)) >1: 1 (eval-case (# # *unspecified*) (else #)) >