The remaining risk, if callers don't respect the contract:
* midori_browser_update_history: * @item: a #KatzeItem * @type: "website", "bookmark" or "download" * @event: "access", "leave", "modify", "delete"
is that KatzeItems of other type are used. For the moment I only think about separators.
It may be more robust for future evolutions to code:
if (!KATZE_ITEM_IS_BOOKMARK (item)) return;
« Back to merge proposal
The remaining risk, if callers don't respect the contract:
* midori_ browser_ update_ history:
* @item: a #KatzeItem
* @type: "website", "bookmark" or "download"
* @event: "access", "leave", "modify", "delete"
is that KatzeItems of other type are used. For the moment I only think about separators.
It may be more robust for future evolutions to code:
if (!KATZE_ ITEM_IS_ BOOKMARK (item))
return;