Menu

#115 Replace Swap with std::swap

*
closed
nobody
None
1
2015-02-11
2014-11-09
No

Not a bug but just a little request: Instead of defining your own Swap function in clipper.cpp, it would be nice if you could replace it with std::swap. Since it's a function template, it works with arbitrary types. I use a slightly modified version of clipper with a different z coordinate type so that the cInt-based Swap doesn't work.
I've attached a patch that replaces the swap function.

1 Attachments

Discussion

  • Angus Johnson

    Angus Johnson - 2014-11-14

    Thanks. Will fix.

     
  • Angus Johnson

    Angus Johnson - 2014-12-16
     
  • Angus Johnson

    Angus Johnson - 2015-02-11
    • status: open --> closed
     

Anonymous
Anonymous

Add attachments
Cancel