Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

List.sort is incorrect (DualPivotQuicksort) #432

Closed
ghost opened this issue Nov 12, 2011 · 3 comments
Closed

List.sort is incorrect (DualPivotQuicksort) #432

ghost opened this issue Nov 12, 2011 · 3 comments
Assignees
@floitschG
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries.

Comments

@ghost
Copy link

ghost commented Nov 12, 2011

LibTest/core/List/sort/List/sort/A01/t04
LibTest/core/List/sort/List/sort/A01/t05
@rakudrama
Copy link
Member

If there is a problem with the sort algorithm I recommend taking the opportunity to change to a stable sort.

@floitschG
Copy link
Contributor

Added Accepted label.

@floitschG
Copy link
Contributor

Fixed in r1506.
Changing sort algorithm is tracked in another bug.

Added Fixed label.

@ghost ghost added Type-Defect area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. labels Nov 14, 2011
@ghost ghost assigned floitschG Nov 14, 2011
nex3 pushed a commit that referenced this issue Aug 31, 2016
@ochafik
Merge pull request #432 from dart-lang/notnull-with-assignments-analysis
3ce73fa 
Less dart.notNull checks for local vars using flow-insensitive nullability inference.
This was referenced Oct 29, 2020
Open
Closed
Closed
This was referenced Nov 5, 2020
Closed
Open
This issue was closed.
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@floitschG floitschG

Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rakudrama @floitschG