Don't search for words smaller than 3 letters

Bug #659234 reported by Jeff Fortin Tam
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Recipe Manager
Fix Committed
Low
Jeff Fortin Tam

Bug Description

I'm not sure how computationally expensive searching/filtering is with large amounts of recipes, but it probably doesn't make sense to start searching until the user has typed at least 3 characters (there are no significant 2-letter words...).

I'm wondering if limiting searches to strings of >=3 characters would boost search/filtering performance, because there would be less matches initially, so less updates to the listview widget, etc.

Tags: performance

Related branches

Revision history for this message
Jeff Fortin Tam (kiddo) wrote :

A possible patch is attached.
The problem is that it causes a traceback when exiting the app:

Exception in thread Thread-1 (most likely raised during interpreter shutdown):
Traceback (most recent call last):
  File "/usr/lib/python2.6/threading.py", line 532, in __bootstrap_inner
  File "/home/jeff/dev/recipe-manager/main/librecipemanager/network.py", line 536, in run
  File "/usr/lib/python2.6/SocketServer.py", line 229, in serve_forever
  File "/usr/lib/python2.6/threading.py", line 380, in set
  File "/usr/lib/python2.6/threading.py", line 291, in notifyAll
<type 'exceptions.TypeError'>: 'NoneType' object is not callable

Revision history for this message
Jeff Fortin Tam (kiddo) wrote :

Nevermind, the traceback was due to bug #758306

Changed in recipe-manager:
status: New → Fix Committed
assignee: nobody → Jean-François Fortin Tam (kiddo)
milestone: none → 0.4
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.