You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Daniel Shahaf <d....@daniel.shahaf.name> on 2011/05/07 11:20:48 UTC

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

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@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