search

ucsb-cs56-projects / cs56-utilities-calculator

-
1 stars 10 forks source link

pull request for proj1 #10

Closed twodimension closed 10 years ago

twodimension commented 10 years ago

@andrewberls -500 points

andrewberls commented 10 years ago

The calculator is looking good! The extra buttons work just like I'd expect and the nested expressions are neat. There are some opportunities for extraction of logic/functionality into variables and methods which is good for clarity and preventing any one section of code from becoming too dense.

andrewberls commented 10 years ago

PROJ1 MOD REVIEW DONE

kjorg50 commented 10 years ago

Closing the pull request at this time, see private feedback repo for details