[PR #159] [MERGED] Use min instead of max #641

New issue

Closed

opened 2026-02-28 14:50:18 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 14:50:18 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Rigellute/spotify-tui/pull/159
Author: @ghost
Created: 11/22/2019
Status: ✅ Merged
Merged: 11/22/2019
Merged by: @Rigellute

Base: master ← Head: revision-search-result

📝 Commits (1)

71771a6 Use min instead of max

📊 Changes

1 file changed (+2 additions, -2 deletions)

View changed files

📝 src/main.rs (+2 -2)

📄 Description

I found a small mistake in my past PR #154.

I used max to get the maximum search limit but it cause an error depending on terminal size because of a rule provided by spotify api.

Here, i use min instead of max and the error does not occur.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Rigellute/spotify-tui/pull/159 **Author:** [@ghost](https://github.com/ghost) **Created:** 11/22/2019 **Status:** ✅ Merged **Merged:** 11/22/2019 **Merged by:** [@Rigellute](https://github.com/Rigellute) **Base:** `master` ← **Head:** `revision-search-result` --- ### 📝 Commits (1) - [`71771a6`](https://github.com/Rigellute/spotify-tui/commit/71771a66d550c43529c7f7434f03a46a6d2ff500) Use min instead of max ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/main.rs` (+2 -2) </details> ### 📄 Description I found a small mistake in my past PR #154. I used `max` to get the maximum search limit but it cause an error depending on terminal size because of a rule provided by spotify api. Here, i use `min` instead of `max` and the error does not occur. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>