[PATCH] Replace explicit version checks with feature checks
Simon Josefsson
simon at josefsson.org
Fri Aug 21 10:34:18 CEST 2009
Jonathan Bastien-Filiatrault <joe at x2a.org> writes:
> Simon Josefsson wrote:
>> Jonathan Bastien-Filiatrault <joe at x2a.org> writes:
>>
>>> Simon Josefsson wrote:
>> How do I create a patch between your branch and master, to review it? I
>> haven't worked with multiple repositories, but willing to learn.
>>
>> /Simon
>
> Alright, here is what you need to do:
>
> - Use a somewhat-pristine git repository cloned from savannah or whatever.
> - Add a remote: git remote add -f jonathan git://x2a.org/gnutls.git ;
> this adds an entry in the .git/config file and fetches refs and objects
> from my repository.
> - Create a local branch tracking my remote repository: git branch
> --track buffers-redux jonathan/buffers-redux
> - Now you can git checkout buffers-redux to switch to that branch or do
> a git diff master..buffers-redux . If you had not created a local branch
> you could instead do git diff master..jonathan/buffers-redux
Excellent, thank you!
Quick comments:
* Please add a comment describing what dump_bytes does, and preferrably
make it use _gnutls_bin2hex for hex encoding. Is there a fixed-size
limit of 128 in the function? If so, it would be nice to fix that.
* What does the 'bufel' variable name refers to? It looks non-english
to me, so I'd prefer using 'buf'.
* Some comments at the top of gnutls_mbuffers.? what it does would be
good.
More later.
/Simon
More information about the Gnutls-devel
mailing list