Hello Greg,

 

now I'm not an authority on this, but just making it return a success code and have a fixme seems fine to me. If you look around, it wouldn't be the first place where this happens either.

Tests won't harm, but then again a success error code is to be expected, IMHO. I'd just send in the patch to make it return TRUE and a success error code for this specific flag.

 

Regards,

Fabian Maurer