Erik Johansson wrote on Wed, Jan 05, 2011 at 22:43:51 +0100: > Hi, > > I have now redone the patch to fix some of the comments given in this > thread and I'm attaching the current working state of it so that we > can have something real to discuss around. This does not include any > changes to svnlook but have a test case to verify that it works as it > should (which it at the moment doesn't, see 10 below). >
Nice work. I can't find a summary anywhere in your mail. Is the summary "Here is a work-in-progress patch that revvs the struct and adds a mod_props member, and below are the specific semantics I'm thinking of for that member; please review those semantics"? Thanks, Daniel > Further more, I've tried to enumerate all cases that I could think of > were properties are changed and how they should be handled. For some > of the cases I've changed my mind compared to what I've said in the > rest of the thread. Hopefully this is a complete list. > > mod_props is the per node hash with properties (see svn_repos_node2_t). > ... snip 12 enumerated items ... > > // Erik > > -- > Erik Johansson > Home Page: http://ejohansson.se/ > PGP Key: http://ejohansson.se/erik.asc
