Forum OpenACS Improvement Proposals (TIPs): Re: Display errors that occurred while sourcing a file in APM package-reload page.

The only thing I don't like about the patch is that it has 1 lock that is used in a lock/unlock cycle for every file that is sourced. It would be better if it could be re-worked to have 1 lock per package. This might mean something like passing errors out of the proc up to the caller so the caller can accumulate the errors into a per-package array/list. The biggest difficulty with that is probably in initializing 1 lock per package (nsv_set apm_load_errors_lock $package_key [ns_mutex create apm_load_errors_${package_key}_lock]).