A Git timeout occured. Only metadata is being displayed

Reviewing merge request #432: Qdbus method overloading support

I added some additional checks to finding a method in qdbus command line tool. Previously, it would take the first method matching the name, so in case of

void foo(int a)
void foo(int a, QStringList b)

and calling

qdbus org.dbustest / foo 15 "(" string list ")"

It would result in "invalid number of parameters" since it tried to match against the first method.

Requested by: Kimmo Kotajärvi
Status: Merged
Source repository: kkos-qt:dbus-overloading
Target repository: qt:master
Created at: January 15th, 2010 10:17

Commits that would be merged:

Version 1

Version 1

Comments
Include comments entered inline

→ State changed from New to Reviewing

Thiago Macieira | 2010-03-05 16:22:44 UTC

→ State changed from Reviewing to Merged

Merged as 0e049a27622a4002af8851fed68f37b71fc7ba54 to 4.7.

Thiago Macieira | 2010-03-05 16:27:11 UTC

Add a new comment:

Login or create an account to post a comment

How to apply this merge request to your repository

One way to merge in these changes is to pull them into a local branch for review, verifying the changes and then merge to back to master.

# Check out a new branch for integration
git checkout -b merge-requests/432

# Fetch the merge request into this branch
git pull \
  git://gitorious.org/qt/qt.git \
  refs/merge-requests/432

# Show the commits, assess they are okay
git log --pretty=oneline --abbrev-commit master..merge-requests/432

# To apply the changes to your branch:
git checkout master
git merge merge-requests/432
git push origin master

Reviewing merge request #432: Qdbus method overloading support

Commits that would be merged:

Comments Include comments entered inline

Add a new comment:

Gitorious

Comments
Include comments entered inline