> -----Ursprüngliche Nachricht----- > Von: Jim Jagielski > > > > That seem to be the case when we have a general error. In other > places where we have a valid 'rv', we tend to return that. > Look at file_cache_recall_mydata() for example... > > In the above, I think the return status may be useful, so > we shouldn't mask it, imo.
As said I am fine with either way, but currently we also return APR_EGENERAL if apr_file_write_full fails below. So we should return in this case rv too. Regards Rüdiger
