Depends on D10032
Details
Diff Detail
- Repository
- rEFL core/efl
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
It seems that this patch has no reviewers specified. If you are unsure who can review your patch, please check this wiki page and see if anyone can be added: https://phab.enlightenment.org/w/maintainers_reviewers/
src/lib/ecore/efl_boolean_model.c | ||
---|---|---|
24 | uhm, this is 16 bit long, so it can have 0..2^16, so i think this comment about 256 is wrong? | |
27 | I think this buffer needs to be bigger ? | |
106 | Are you sure about the & in the sizeof? | |
108 | I think this here is a problem, the default value is either 0 or 1 but memset does set things bytewise. | |
462–463 | Isnt that wrong and we should only do that if the first bit is equal to request earlier on ? | |
469 | I think all default values in the iterator have to be the same as we are walking the same boolean property type, why is this check here? |
src/lib/ecore/efl_boolean_model.c | ||
---|---|---|
27 | Nah the size is correct. Problem with 256, is that if you use a uchar to count, you will wrap in some case without a way to detect it (So for safety I switched to ushort). Will change the comment accordingly. | |
106 | I will double check that. I did make a few change along the way and it is indeed not clear this is correct. | |
108 | default_value is set via !! which force a full bytewise 0 and 1 to be set. | |
462–463 | You are right the init case is not properly handle. Need to make sure that at least there is one first run being done before returning an index. | |
469 | You can request an iterator on all value being TRUE or FALSE. As the default value can also be one or the other, we need to stop walking if the default value is not the same as the requested value for the iterator. |
src/lib/ecore/efl_boolean_model.c | ||
---|---|---|
108 | Oh, that is new black magic to me. |