Two minor fixes #56

Open
wants to merge 3 commits into
from

Conversation

Projects
None yet
1 participant

franzen commented Apr 14, 2012

Hi, here comes two minor fixes to your code:

  1. Allow ()/ in query strings to allow grouping and things like NEAR/5
  2. Don't append empty 'where' conditions, since it later will fail with a:
    Wrong arguments for overloaded method 'Query.new'.
    Possible C/C++ prototypes are:
    Query.new(Xapian::Query const &copyme)
    Query.new()
    Query.new(std::string const &tname_, Xapian::termcount wqf_, Xapian::termpos pos_)
    Query.new(std::string const &tname_, Xapian::termcount wqf_)
    Query.new(std::string const &tname_)
    Query.new(Xapian::Query::op op_, Xapian::Query const &left, Xapian::Query const &right)
    Query.new(Xapian::Query::op op_, std::string const &left, std::string const &right)
    Query.new(Xapian::Query::op op_, Xapian::Query q, double parameter)
    Query.new(Xapian::Query::op op_, Xapian::valueno slot, std::string const &begin, std::string const &end)
    Query.new(Xapian::Query::op op_, Xapian::valueno slot, std::string const &value)
    Query.new(Xapian::PostingSource *external_source)
    Query.new(Xapian::Query::op op, std::vector< Xapian::Query > const &subqs, Xapian::termcount param)
    Query.new(Xapian::Query::op op, std::vector< Xapian::Query > const &subqs)

Nils Franzen added some commits Apr 14, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment