You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@subversion.apache.org by Julian Foad <ju...@btopenworld.com> on 2012/12/21 22:05:57 UTC

WC DB 'kind' column value needed for a base-deleted row?

I just noticed that we insert a node kind into a 'base-deleted' row, at least in some cases.

wc_db.c: svn_wc__db_extend_parent_delete(... kind ...):  stmt = STMT_INSTALL_WORKING_NODE_FOR_DELETE

  svn_sqlite__bindf(... kind ...)

Is there a good reason for this?  It makes me feel that it is redundant and therefore likely to get forgotton, get out of sync and lead to hard-to-discover errors.

Does anyone have an objection to me making the rule be that the 'kind' field should be left as null when presence==base-deleted, and figuring out what fixes we need to make to the code to implement that?

- Julian