← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to HACKING

  • Committer: Wouter van Heyst
  • Date: 2006-06-07 11:13:02 UTC
  • mfrom: (1747 +trunk)
  • mto: This revision was merged to the branch mainline in revision 1752.
  • Revision ID: larstiq@larstiq.dyndns.org-20060607111302-bce5a90c86f0d764
[merge] bzr.dev 1747

Show diffs side-by-side

added added

removed removed

Lines of Context:
HACKING
194
194
    may not catch every case but it's still useful sometimes.
195
195
 
196
196
 
 
197
Factories
 
198
---------
 
199
 
 
200
In some places we have variables which point to callables that construct
 
201
new instances.  That is to say, they can be used a lot like class objects,
 
202
but they shouldn't be *named* like classes:
 
203
 
 
204
> I think that things named FooBar should create instances of FooBar when
 
205
> called. Its plain confusing for them to do otherwise. When we have
 
206
> something that is going to be used as a class - that is, checked for via
 
207
> isinstance or other such idioms, them I would call it foo_class, so that
 
208
> it is clear that a callable is not sufficient. If it is only used as a
 
209
> factory, then yes, foo_factory is what I would use.
 
210
 
 
211
 
197
212
Writing output
198
213
==============
199
214