COIN-OR::LEMON - Graph Library

Opened 11 years ago

Closed 11 years ago

#127 closed enhancement (fixed)

Uniform interface for *ArcLookUp

Reported by: Alpar Juttner Owned by: Balazs Dezso
Priority: major Milestone: LEMON 1.0 release
Component: core Version: hg main
Keywords: Cc:
Revision id:

Description

The classes AllArcLookUp and DynArcLookUp have (almost) the same functionality, but they have different APIs for listing the parallel edges. A uniform interface would be much better.

Tic ticket refers to [b67149f0e675]

Attachments (3)

0915721396dc.patch (7.9 KB) - added by Balazs Dezso 11 years ago.
e39056157d24.patch (1.1 KB) - added by Balazs Dezso 11 years ago.
28239207a8a3.patch (7.2 KB) - added by Balazs Dezso 11 years ago.

Download all attachments as: .zip

Change History (7)

Changed 11 years ago by Balazs Dezso

Attachment: 0915721396dc.patch added

comment:1 Changed 11 years ago by Balazs Dezso

Owner: changed from Alpar Juttner to Balazs Dezso
Status: newassigned

I have changed the interface in the [0915721396dc], and I have found two bugs, which are solved also.

comment:2 in reply to:  1 ; Changed 11 years ago by Alpar Juttner

Replying to deba:

I have changed the interface in the [0915721396dc], and I have found two bugs, which are solved also.

I found a lot of irrelevant white-space changes in this patch.

Changed 11 years ago by Balazs Dezso

Attachment: e39056157d24.patch added

Changed 11 years ago by Balazs Dezso

Attachment: 28239207a8a3.patch added

comment:3 in reply to:  2 ; Changed 11 years ago by Balazs Dezso

Replying to alpar:

Replying to deba:

I have changed the interface in the [0915721396dc], and I have found two bugs, which are solved also.

I found a lot of irrelevant white-space changes in this patch.

I reverted the whitespace changes. I also split the changeset to two parts, the [e39056157d24] patch contains the bug fixes, while the [28239207a8a3] patch contains the interface change.

comment:4 in reply to:  3 Changed 11 years ago by Alpar Juttner

Resolution: fixed
Status: assignedclosed

Replying to deba:

I reverted the whitespace changes. I also split the changeset to two parts, the [e39056157d24] patch contains the bug fixes, while the [28239207a8a3] patch contains the interface change.

Thanks. The changes are now in the main branch.

Note: See TracTickets for help on using tickets.