marcacohen/boto — comment on commit 62eb29d3d510b943a31ec2084d9ac45910d92fe3
File: boto/s3/bucket.py line 622
By mfschwartz:

Since set_def_xml_acl is only defined in the gs-specific subclass I think it would be cleaner to leave that example out of this line of documentation. And in fact I have a question about whether it really makes sense to define this abstraction in the parent class. It seems the reason to do this is to use subclassing for sharing code, which I think in general is not a good use of subclassing. How about putting this function in the gs-specific subclass (and not worrying about making it share code with the s3 class) or moving it to a utility class?