Code review comment for lp:~jobh/dolfin/symmetric-assemble

I'm not convinced that this is the best approach. I'm inclined towards supporting unassembled matrices, and using this to store the cell tensors.

With the code, I'd like to see more verbose variable naming. It's too terse for me.

I wouldn't back port this. It's and feature and not a bug. I would prefer to make new releases to get new functionality out.