[ET Trac] [Einstein Toolkit] #1589: suspicious code for operator+ in bboxset2

Einstein Toolkit trac-noreply at einsteintoolkit.org
Thu Apr 24 14:37:38 CDT 2014


#1589: suspicious code for operator+ in bboxset2
---------------------+------------------------------------------------------
  Reporter:  rhaas   |       Owner:  eschnett           
      Type:  defect  |      Status:  closed             
  Priority:  major   |   Milestone:                     
 Component:  Carpet  |     Version:  development version
Resolution:  fixed   |    Keywords:  Carpet             
---------------------+------------------------------------------------------

Comment (by eschnett):

 A correction, for the record -- I misspoke slightly.

 {{{operator+}}} is not the symmetric set difference, this is
 {{{operator^}}}. {{{operator+}}} is the set union (similar to
 {{{operator|}}}), but the caller guarantees that the sets are disjoint,
 which allows certain optimizations. In this case, {{{operator|}}} is the
 same as {{{operator^}}}, so we can use the (here much cheaper)
 {{{operator^}}} to implement {{{operator+}}}.

-- 
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1589#comment:6>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit


More information about the Trac mailing list