Skip to content

handle sort(reverse=True) keyword argument#38

Merged
medvednikov merged 1 commit intovlang:masterfrom
ylluminate:fix-sort-reverse
Feb 20, 2026
Merged

handle sort(reverse=True) keyword argument#38
medvednikov merged 1 commit intovlang:masterfrom
ylluminate:fix-sort-reverse

Conversation

@ylluminate
Copy link
Contributor

Python's list.sort(reverse=True) was transpiled to sort(a < b), ignoring the reverse keyword entirely. Now checks keyword arguments and uses sort(a > b) for descending order.

Changes

  • visit_call() in transpiler.v: Inspect node.keywords for reverse=True, emit sort(a > b) for descending order
  • 1 test expected output updated (list_sort)

All 108 tests pass.

Python's list.sort(reverse=True) was transpiled to sort(a < b), ignoring
the reverse keyword entirely. Now checks keyword arguments and uses
sort(a > b) for descending order.

All 108 tests pass.
@medvednikov medvednikov merged commit 7c41a99 into vlang:master Feb 20, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants