[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Expose svn_info_t->depth to JavaHL

From: Mark Phippard <markphip_at_gmail.com>
Date: Tue, 1 Jul 2008 08:26:28 -0400

On Tue, Jul 1, 2008 at 12:31 AM, Hyrum K. Wright
<hyrum_wright_at_mail.utexas.edu> wrote:
> Jeremy Whitlock wrote:
>>
>> Hi All,
>> Attached is a patch that will expose svn_info_t->depth to JavaHL's
>> Info2 class. This enhancement will enable me to follow up with
>> another patch that will enable JavaHL to check whether a working copy
>> path is complete, or not a sparse checkout. (This is something that
>> could be used to enhance Subclipse's best practices checks for
>> example.) For quick analysis, I have pasted the log message at the
>> bottom of this email. Please let me know if there is anything I can
>> do to make this patch ready for application.
>>
>> Take care,
>>
>> Jeremy
>>
>> [[[
>> Expose the depth member of svn_info_t to JavaHL's Info2 class.
>>
>> *
>> subversion/bindings/javahl/tests/org/tigris/subversion/javahl/BasicTests.java
>> (testBasicInfo2): Updated tests to check depth.
>>
>> * subversion/bindings/javahl/native/EnumMapper.h
>> (mapDepth): New function.
>>
>> * subversion/bindings/javahl/native/EnumMapper.cpp
>> (mapDepth): Implementation.
>>
>> * subversion/bindings/javahl/native/InfoCallback.cpp
>> (createJavaInfo2): Updated to call new depth-aware constructor.
>>
>> * subversion/bindings/javahl/src/org/tigris/subversion/javahl/Info2.java
>> (Info2): Updated constructor for depth argument.
>> (getDepth, setDepth): New functions.
>> ]]]
>
> Jeremy,
> Thanks for the patch. I tweaked the log message just a tad and committed
> this in r31936.
>
> I noticed that this patch had the mime-type of application/octet-stream.
> That makes it a bit difficult to review inline. Would it be possible for
> you to sent future patches as text/x-diff, text/x-patch, or text/plain?

Does anyone know how to do that with GMail? I have never figured it
out, and it always uses application/octet-stream when I attach a
patch.

-- 
Thanks
Mark Phippard
http://markphip.blogspot.com/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-07-01 14:26:51 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.