[PR #1265] [MERGED] changed list length to reflect contents of the list #1864

New Issue

claunia · 2026-01-29T15:25:31Z

claunia commented

2026-01-29 15:25:31 +00:00

📋 Pull Request Information

Original PR: https://github.com/TheAlgorithms/C/pull/1265
Author: @aracnoid149
Created: 6/2/2023
Status: ✅ Merged
Merged: 6/20/2023
Merged by: @Panquesito7

Base: master ← Head: patch-1

📝 Commits (3)

b2365ee changed list length to reflect contents of the list
830f450 Merge branch 'master' into patch-1
4df8a69 Merge branch 'master' into patch-1

📊 Changes

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

View changed files

📝 data_structures/list/list.c (+2 -2)

📄 Description

I changed the return value of n in List_length to reflect the number of items inside the list, so a newly initialized list will return a length of 0. To prevent items in List_toArray from being cut off, I added one back to n at the beginning of the List_toArray function.

Description of Change

References

Checklist

Added description of change
Added file name matches File name guidelines
Added tests and example, test must pass
Relevant documentation/comments is changed or added
PR title follows semantic commit guidelines
Search previous suggestions before making a new one, as yours may be a duplicate.
I acknowledge that all my contributions will be made under the project's license.

Notes:

_{🔄 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/TheAlgorithms/C/pull/1265 **Author:** [@aracnoid149](https://github.com/aracnoid149) **Created:** 6/2/2023 **Status:** ✅ Merged **Merged:** 6/20/2023 **Merged by:** [@Panquesito7](https://github.com/Panquesito7) **Base:** `master` ← **Head:** `patch-1` --- ### 📝 Commits (3) - [`b2365ee`](https://github.com/TheAlgorithms/C/commit/b2365ee566945670d3ca731f76e5ffd3e882a2e7) changed list length to reflect contents of the list - [`830f450`](https://github.com/TheAlgorithms/C/commit/830f45001aa9a27d91c24138eb66001c205a2d6a) Merge branch 'master' into patch-1 - [`4df8a69`](https://github.com/TheAlgorithms/C/commit/4df8a697f1ba9dbe1679ee872d06ff1989b7f806) Merge branch 'master' into patch-1 ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `data_structures/list/list.c` (+2 -2) </details> ### 📄 Description I changed the return value of n in List_length to reflect the number of items inside the list, so a newly initialized list will return a length of 0. To prevent items in List_toArray from being cut off, I added one back to n at the beginning of the List_toArray function. #### Description of Change  #### References  #### Checklist  - [ ] Added description of change - [ ] Added file name matches [File name guidelines](https://github.com/TheAlgorithms/C/blob/master/CONTRIBUTING.md#File-Name-guidelines) - [ ] Added tests and example, test must pass - [ ] Relevant documentation/comments is changed or added - [ ] PR title follows semantic [commit guidelines](https://github.com/TheAlgorithms/C/blob/master/CONTRIBUTING.md#Commit-Guidelines) - [x] Search previous suggestions before making a new one, as yours may be a duplicate. - [x] I acknowledge that all my contributions will be made under the project's license. Notes:  --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

claunia added the pull-request label 2026-01-29 15:25:31 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/C#1864