Fix prototype.constructor.

d3 / d3-shape

Graphical primitives for visualization, such as lines and areas.

https://d3js.org/d3-shape

ISC License

2.47k stars 307 forks source link

Fix prototype.constructor. #59

Closed mbostock closed 8 years ago

mbostock commented 8 years ago

The current definition obliterates the constructor property.

[ ] Linear
[ ] Radial
[ ] Basis
[ ] BasisClosed
[ ] BasisOpen
[ ] Bundle
[ ] Cardinal
[ ] CardinalClosed
[ ] CardinalOpen
[ ] CatmullRom
[ ] CatmullRomClosed
[ ] CatmullRomOpen
[ ] LinearClosed
[ ] MonotoneX
[ ] MonotoneY
[ ] Natural
[ ] Step

mbostock commented 8 years ago

Actually, I think this is fine; these classes are anonymous, and I don’t see a strong reason to support instanceof, prototype extension, and constructor inspection.