[flashrom-gerrit] Change in flashrom[master]: libflashrom,writeprotect: add flashrom_wp_get_available_ranges()

[Nikolai Artemiev (Code Review)]

Attention is currently required from: Nico Huber, Edward O'Callaghan, Angel Pons, Anastasia Klimchuk, Sergii Dmytruk.
Nikolai Artemiev has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/58481 )

Change subject: libflashrom,writeprotect: add flashrom_wp_get_available_ranges()
......................................................................

Patch Set 11:

(3 comments)

File writeprotect.c:

https://review.coreboot.org/c/flashrom/+/58481/comment/dee3641b_00feabce 
PS10, Line 271: ranges[++last] = ranges[i];
> Thank you! […]

Done

https://review.coreboot.org/c/flashrom/+/58481/comment/fec2ffd6_8b09854f 
PS10, Line 271: ranges[++last] = ranges[i];
> Thank you! […]

I've rewritten the loop to avoid edge cases and be easier to read.

Probably out of scope for now, but maybe should we have a separate file for collecting common algorithms like this (sort, binary search, deduplicate, etc) instead of repeating them in various places?

https://review.coreboot.org/c/flashrom/+/58481/comment/8721f486_2d9a7be3 
PS10, Line 274: 	*count = last + 1;
> If `*count` was 0 initially, i.e. an empty list, this would set it to `1`.

Done