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
Received on 2012-12-21 22:06:35 CET