guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#26940: [PATCH 1/3] gnu: Add http-parser.


From: Ricardo Wurmus
Subject: bug#26940: [PATCH 1/3] gnu: Add http-parser.
Date: Mon, 15 May 2017 21:26:09 +0200
User-agent: mu4e 0.9.18; emacs 25.2.1

Marius Bakke <address@hidden> writes:

> Ricardo Wurmus <address@hidden> writes:
>
>> * gnu/packages/web.scm (http-parser): New variable.
>
> [...]
>
>> +    (description "This is a parser for HTTP messages written in C.  It 
>> parses
>> +both requests and responses. The parser is designed to be used in 
>> performance
>> +HTTP applications. It does not make any syscalls nor allocations, it does 
>> not
>> +buffer data, it can be interrupted at anytime. Depending on your 
>> architecture,
>> +it only requires about 40 bytes of data per message stream (in a web server
>> +that is per connection).")
>
> Some missing spaces in the description here. The sentences also read a
> bit weird, should it be "high-performance HTTP applications"? Or
> performant? Maybe just "fast"? LGTM with cosmetic fix.

Thanks for the suggestions!

--
Ricardo

GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
https://elephly.net






reply via email to

[Prev in Thread] Current Thread [Next in Thread]