Don`t mutate options in the linter

Review Request #1978 - Created March 23, 2015 and submitted

Information
Stu Hood
pants
ac9a37d...
Reviewers
pants-reviews
benjyw, nhoward_tw, tejal
  • Don`t mutate options, otherwise we end up growing this list in memory. Noticed this because scrooge complains if it receives the same option twice.

Rather than writing a test specific to options on thrift_linter, I'd like to follow up by making option values immutable (at the top level at least), to prevent any other bugs like this.


  
Ity Kaul
Nick Howard (Twitter)
Jin Feng
Stu Hood
Review request changed

Status: Closed (submitted)

Loading...