On Wed, Aug 7, 2013 at 10:47 AM,  <[email protected]> wrote:
> Revision: 56668
>           http://sourceforge.net/p/brlcad/code/56668
> Author:   starseeker
...
> Modified Paths:
> --------------
>     brlcad/trunk/src/conv/step/ON_Brep.cpp
>
> Modified: brlcad/trunk/src/conv/step/ON_Brep.cpp
> ===================================================================
> --- brlcad/trunk/src/conv/step/ON_Brep.cpp      2013-08-07 15:16:35 UTC (rev 
> 56667)
> +++ brlcad/trunk/src/conv/step/ON_Brep.cpp      2013-08-07 15:47:22 UTC (rev 
> 56668)
> @@ -62,25 +62,36 @@
>
>  bool ON_BRep_to_STEP(ON_Brep *brep, Registry *registry, InstMgr 
> *instance_list)
> -       STEPentity ** vertex_pt_array = new STEPentity*[brep->m_V.Count()];
...
> +       std::vector<STEPentity *> vertex_pnts;
...
> +       /* Set initial container capacities */
...
> +       vertex_pnts.resize(brep->m_V.Count(), NULL);

Better to use one step construction (and zero instead of NULL per
Scott Meyers)?:

  std::vector<STEPentity *> vertex_pnts(brep->m_V.Count(), 0);

Best,

-Tom

------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite!
It's a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead. 
Download for free and get started troubleshooting in minutes. 
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk
_______________________________________________
BRL-CAD Developer mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-devel

Reply via email to