gnutls-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] Replace explicit version checks with feature checks


From: Simon Josefsson
Subject: Re: [PATCH] Replace explicit version checks with feature checks
Date: Fri, 21 Aug 2009 10:34:18 +0200
User-agent: Gnus/5.110011 (No Gnus v0.11) Emacs/23.1 (gnu/linux)

Jonathan Bastien-Filiatrault <address@hidden> writes:

> Simon Josefsson wrote:
>> Jonathan Bastien-Filiatrault <address@hidden> 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




reply via email to

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