Hi Lars,

So in theory this could be written like:
int max = -1;
iter_open_list(&iter);

while (iter_next_range(&iter, &max1, &max2)) {
        if (max2 > max)
                max = max2;
}

iter_close_list(&iter);

I can do this and it does indeed work! :-)

This may be my personal thing, but I find this not very intuitive and
hard to read.

To make this more intuitive can we:
* move your proposed while loop to a new function
* name that function ...iter_next_range_or_list (or something like that)
* put that function into gatresult.c

I'm confused. iter_next_range specifically handles ranges as defined by ITU v.250. See section 5.7.3.1 "Range of Values".

Here's a small quote:
"If more than one value is supported, then the values may be listed individually, separated by comma characters (IA5 2/12), or, when a continuous range of values is supported, by the first value in the range, followed by a hyphen character (IA5 2/13), followed by the last value in the range. The specification of single values and ranges of values may be intermixed within a single information text. In all cases, the supported values shall be indicated in ascending order."

The only thing the user has to do is open up a list, since some modems get this wrong and don't enclose the entire set in '()'

* and in the new at_cgerep_test_cb just use this function ?

How would you return the set of supported settings? I guess you could use something like l_uintset, but even knowing the valid theoretical range is tricky.


I can imagine this new function can be of use at other places as well.
What do you think ?

Given my confusion above, no idea?

Regards,
-Denis
_______________________________________________
ofono mailing list -- ofono@ofono.org
To unsubscribe send an email to ofono-le...@ofono.org

Reply via email to