Re: [ptp-dev] rdt.sync.ui

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]

Re: [ptp-dev] rdt.sync.ui

From: "Albert L. Rossi" <arossi@xxxxxxxxxxxxx>
Date: Mon, 10 Jan 2011 11:09:52 -0600 (CST)
Delivered-to: ptp-dev@xxxxxxxxxxx
List-archive: <https://dev.eclipse.org/mailman/private/ptp-dev>
List-help: <mailto:ptp-dev-request@eclipse.org?subject=help>
List-subscribe: <https://dev.eclipse.org/mailman/listinfo/ptp-dev>, <mailto:ptp-dev-request@eclipse.org?subject=subscribe>
List-unsubscribe: <https://dev.eclipse.org/mailman/options/ptp-dev>, <mailto:ptp-dev-request@eclipse.org?subject=unsubscribe>

Hi,

I just checked out a fresh copy of HEAD, and noticed an API inconsistency in org.eclipse.ptp.rdt.sync.ui

org.eclipse.ptp.internal.rdt.sync.ui.scannerinfo.RemoteIncludeDialog, line 175
org.eclipse.ptp.rdt.sync.ui.wizards.RemoteProjectContentsLocationArea, line 244

call a method from org.eclipse.ptp.remote.ui.IRemoteConnection, openConnectionWithProgress, whose signature requires org.eclipse.jface.operation.IRunnableContext as its second of three arguments, but evidently there was a previous version with two arguments, and that is the one being called at those points.

Al

Follow-Ups:
- Re: [ptp-dev] rdt.sync.ui
  - From: Roland Schulz

References:
- [ptp-dev] new rm implementation
  - From: Greg Watson

Prev by Date: [ptp-dev] CORRECTION: Monthly call for PTP users: 4th Wednesday 2pm ET
Next by Date: Re: [ptp-dev] new rm implementation
Previous by thread: [ptp-dev] new rm implementation
Next by thread: Re: [ptp-dev] rdt.sync.ui
Index(es):
- Date
- Thread

Breadcrumbs