Fix implementation of shallow copy on OptionValueContainer.

Review Request #3041 — Created Oct. 26, 2015 and submitted

benjyw
pants
75b5586...
pants-reviews
stuhood
- Adds a test to verify it.
- Uses it instead of an unnecessary deepcopy.

Those deepcopies were significant enough to show up in profiles.

CI passes: https://travis-ci.org/pantsbuild/pants/builds/87528183.

BE
  1. Ping?

  2. 
      
ST
  1. Ship It!
  2. 
      
NH
  1. Ship It!
  2. 
      
BE
  1. Thanks! Submitted as c1684754289af66fb8e9d12dbff181c9320961ec.

  2. 
      
BE
Review request changed

Status: Closed (submitted)

Change Summary:

c1684754289af66fb8e9d12dbff181c9320961ec

Loading...