From d9636fdbc8fecd7660e91ff3300d468c9d54d470 Mon Sep 17 00:00:00 2001
From: Chris Oelmueller <chris.oelmueller@gmail.com>
Date: Wed, 21 Aug 2013 16:50:48 +0200
Subject: Shopping list: Do not prompt for removing manuals

As the comment said, but the logic not really supported, those are
consumables and as such it is very feasible to continue being interested
in them after identifying one of their kind.

Tomes of destruction technically also fit that description, but instead
of prompting for removal they are just automatically removed now when
another tome is encountered. Whyever they ended up on a shopping list at
all will remain a mystery though.
---
 crawl-ref/source/shopping.cc | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/crawl-ref/source/shopping.cc b/crawl-ref/source/shopping.cc
index 23ad12f..4ab16a4 100644
--- a/crawl-ref/source/shopping.cc
+++ b/crawl-ref/source/shopping.cc
@@ -2698,16 +2698,16 @@ unsigned int ShoppingList::cull_identical_items(const item_def& item,
             return 0;
     }
 
+    // Manuals are consumable, and interesting enough to keep on list.
+    if (item.base_type == OBJ_BOOKS && item.sub_type == BOOK_MANUAL)
+            return 0;
+
     // Item is already on shopping-list.
     const bool on_list = find_thing(item, level_pos::current()) != -1;
 
     const bool do_prompt =
         (item.base_type == OBJ_JEWELLERY && !jewellery_is_amulet(item)
-         && ring_has_stackable_effect(item))
-     // Manuals and tomes of destruction are consumable.
-     || (item.base_type == OBJ_BOOKS
-         && (item.sub_type == BOOK_MANUAL
-             || item.sub_type == BOOK_DESTRUCTION));
+         && ring_has_stackable_effect(item));
 
     bool add_item = false;
 
-- 
1.8.3.4