John-Mark Bell wrote: >> for a very similar reason to the reason that we send a browser_window >> rather than a search_context; the search_context could be NULL [the >> sign that the search is a new search], so the function needs the >> necessary references in case we're creating a new search_context >> > Right. Can you remind me why it's this way, rather than the frontend > creating/destroying the search context as appropriate? > right :-) for the sake of efficiency, as we already discussed in irc;
one check in the core function translates to multiple checks in the front functions before the calls; after all, one of the main reasons for moving code to the core is that it avoids unnecessary repetition in the fronts Best Mark
