On Wed, Jun 15, 2011 at 06:55:19AM +0300, Daniel Shahaf wrote: > Daniel Shahaf wrote on Wed, Jun 15, 2011 at 06:33:46 +0300: > > s...@apache.org wrote on Tue, Jun 14, 2011 at 15:10:50 -0000: > > > + svn_stringbuf_appendcstr(buf, _("Local property value:\n")); > > > + if (mine_is_binary) > > > + svn_stringbuf_appendcstr(buf, _("Cannot display: property value > > > is " > > > + "binary data\n")); > > > > Could we just print a hex dump of the value? > > Concretely: > > [[[ > but the property has already been locally changed to a different value. > Local property value: > mine > Incoming property value: > 0x7E454B4602010100000000000000000002003D0001000000A01440000000000040000000000000 > ]]]
While I understand your reasoning I think this would be very confusing to many users. Not all SVN users are programmers. And I guess people will in general be using special tools anyway to view such properties. E.g. if the prop value was a PNG image, they will want to see the image. A string of hex numbers won't help them much. There is precedence here, too. We don't show diffs of binary files, we show a placeholder instead that says a diff cannot be shown. Would you also prefer svn diff to print a diff of the hex dumps? I'd doubt it. I'd say let them propget the value and look at it in whatever manner they see fit. This is a total edge case anyway. No user using standard svn: properties is ever going to run into this. > Patch to produce that: