[email protected] wrote:
This looks fine to me.  I find the variable naming in this function to
be a bit confusing; insert_path and legacy_path might be better
replacements for src_path and cur_full_path, but I realize you didn't
write this code.  I understand if you don't want to change these now.

I'd prefer to to leave it as is for now, especially since this particular nomenclature is used throughout the file_manager.

Cheers,
--
Shawn Walker
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to