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

Re: svn commit: r1095181 - in /subversion/trunk/subversion: svn/notify.c svn/propdel-cmd.c tests/cmdline/prop_tests.py

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Sat, 7 May 2011 12:20:48 +0300

Coming back to this three weeks later: I wanted to understand what the
change was, and the first three paragraphs don't tell me that.

hwright_at_apache.org wrote on Tue, Apr 19, 2011 at 19:51:18 -0000:
> Author: hwright
> Date: Tue Apr 19 19:51:18 2011
> New Revision: 1095181
>
> URL: http://svn.apache.org/viewvc?rev=1095181&view=rev
> Log:
> Ah, the foibles of youth.
>
> Two years ago, I committed r875310 which fixed issue #2220:
> "'svn propdel' returns success on deleting a non-existent prop". It turns out
> that my younger, less-experienced self was just making life hard for the
> slightly more gray-haired and battle-hardened self I am today (no doubt caused
> by code such as this).
>
> So, in homage to the ever-continuing course of self-improvement (and in an
> effort to over-use and even abuse the punctuation mark know as the hyphen), I
> give you this revision, which simplifies, clarifies, and improve-ifies the
> non-existent property deletion notification.
>
> Just use the standard notification infrastructure to when we attempt to
> delete a nonexistent property.
>
> * subversion/tests/cmdline/prop_tests.py
Received on 2011-05-07 11:21:48 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.