Encapsulate the details.

Moved Completion::getCompletion() in ICompletionCollector and made the filtering of completion items more C++ friendly.

By setting the window and button text colors to the tool tip text color,
since a tool tip background is drawn.

Task-number: QTCREATORBUG-322

When constructor completion was attempted on a base class specified in a
class declaration, a null pointer reference would occur.

Task-number: QTCREATORBUG-321
Reviewed-by: Christian Kamm <christian.d.kamm@nokia.com>

Done with Erik Verbruggen

They should always use the function parameter tooltip.

This fixes a bug where you were offered completion for
C foo( -> C foo(int x)
if C had a constructor taking int x.

Reviewed-by: Thorbjørn Lindeijer <thorbjorn.lindeijer@nokia.com>

Reviewed-by: Thorbjørn Lindeijer

Instead of just completing
void A::foo(|) -> void A::foo(int i|), we now complete
void A::foo(|) -> void A::foo(int i) const|
where | represents the place of the cursor.

Reviewed-by: Thorbjørn Lindeijer <thorbjorn.lindeijer@nokia.com>

Reviewed-by: Roberto Raggi <roberto.raggi@nokia.com>

You now get the function parameters as a completion suggestion when
declaring or defining a function. The tooltip for function calls
is unchanged.

Reviewed-by: Roberto Raggi <roberto.raggi@nokia.com>
(cherry picked from commit 202fef42)

You now get the function parameters as a completion suggestion when
declaring or defining a function. The tooltip for function calls
is unchanged.

Reviewed-by: Roberto Raggi <roberto.raggi@nokia.com>

Share code with ResolveExpression.

struct foo {
  int m_zoo;

  foo(): m_ // cursor is here.

Which is 'quite' important, since Qt builds by default generate
frameworks.

Need to check the character to the right of the cursor.

Also skip semicolons in when auto parentheses insertion is enabled.

Done with mae.

The completion for the directory wasn't triggered because one of the
items matched the typed string, so the list of completion wasn't empty
and no new trigger was allowed.

By not matching items when a trailing slash is typed, the list is
allowed to trigger again from the new location.

Reviewed-by: Daniel Molkentin <daniel.molkentin@nokia.com>

When a closing character is already there, it shouldn't be appended.
Done by generalizing the same code for automatically inserted brackets
after C++ symbols.