When bug 324994 is done, storage will call SetIsVoid on null strings so we won't have to. Remove the checks in places for this.
Created attachment 211068 [details] [diff] [review] Patch Now that the storage service sets IsVoid when the column is NULL, we can remove our checks.
In testing this it seems sqlite treats empty strings the same as NULLs so this doesn't do what we want. Instead, we should have another method for differentiating unset user titles from set empty user titles.
My previous tests were wrong, null and empty string are distinct
Created attachment 211093 [details] [diff] [review] New patch This patch is the uploaded one. It fixes a one-line problem where void strings are not handled properly in row reading.
On branch and trunk
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h". In Thunderbird 3.0b, you do that as follows: Tools | Message Filters Make sure the correct account is selected. Click "New" Conditions: Body contains places-to-b-and-h Change the action to "Delete Message". Select "Manually Run" from the dropdown at the top. Click OK. Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter. Gerv