emo/xbps
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

xbps_array_foreach_cb_multi: improve previous (v2).

Browse Source 
As suggested by @CasperVector reuse the 'i' var to
wait for threads that were created successfully.
This commit is contained in:
Juan RP 2019-12-27 15:47:43 +01:00
parent a3a1c372cb
commit f5d93caf15
No known key found for this signature in database
GPG Key ID: AF19F6CB482F9368
1 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
lib/plist.c
View File

@ -102,7 +102,7 @@ xbps_array_foreach_cb_multi(struct xbps_handle *xhp,
{
struct thread_data *thd;
unsigned int arraycount, slicecount;
int rv = 0, error = 0 , maxthreads;
int rv = 0, error = 0, i, maxthreads;
unsigned int reserved;
pthread_spinlock_t reserved_lock;
@ -138,7 +138,7 @@ xbps_array_foreach_cb_multi(struct xbps_handle *xhp,
reserved = slicecount * maxthreads;
for (int i = 0; i < maxthreads; i++) {
for (i = 0; i < maxthreads; i++) {
thd[i].array = array;
thd[i].dict = dict;
thd[i].xhp = xhp;
@ -156,9 +156,11 @@ xbps_array_foreach_cb_multi(struct xbps_handle *xhp,
}
}
/* wait for all threads */
for (int i = 0; i < maxthreads; i++)
rv = pthread_join(thd[i].thread, NULL);
/* wait for all threads that were created successfully */
for (int c = 0; c < i; c++) {
if ((rv = pthread_join(thd[c].thread, NULL)) != 0)
error = rv;
}
out:
free(thd);