Thanks to visit codestin.com
Credit goes to github.com

Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: rdpeng/ProgrammingAssignment2
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Choose a base ref
...
head repository: wcooley/ProgrammingAssignment2
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: master
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 7 commits
  • 2 files changed
  • 1 contributor

Commits on May 24, 2014

  1. Add basic working implementations

    @wcooley
    wcooley committed May 24, 2014
    Configuration menu
    Copy the full SHA
    93cc08e View commit details
    Browse the repository at this point in the history

Commits on May 25, 2014

  1. Add test driver

    @wcooley
    wcooley committed May 25, 2014
    Configuration menu
    Copy the full SHA
    bb048d7 View commit details
    Browse the repository at this point in the history

  2. Add some explanatory comments

    @wcooley
    wcooley committed May 25, 2014
    Configuration menu
    Copy the full SHA
    6a7688e View commit details
    Browse the repository at this point in the history

  3. Refactor lots of the internals 

    * Fix bug that cache wasn't being invalidated due to using `m` instead
  of `invmatrix` (now `altx).

* Reflavor the cache to be about an alternative representation of the
  matrix, rather than necessarily just inversion.

* Add message indicating when cache is being invalidated.
    @wcooley
    wcooley committed May 25, 2014
    Configuration menu
    Copy the full SHA
    da022a3 View commit details
    Browse the repository at this point in the history

  4. Move both branches w/in conditional for symmetry 

    and to have a single exit from the function.
    @wcooley
    wcooley committed May 25, 2014
    Configuration menu
    Copy the full SHA
    8870147 View commit details
    Browse the repository at this point in the history

  5. Use set_alt to in set instead of a 2nd assignment

    @wcooley
    wcooley committed May 25, 2014
    Configuration menu
    Copy the full SHA
    11a3ba5 View commit details
    Browse the repository at this point in the history

  6. Use multiline block for set_alt 

    I am not crazy about single-statement-non-block for anything
non-trivial; it makes it to easy to miss things (see the Apple "goto
fail" SSL bug for example).
    @wcooley
    wcooley committed May 25, 2014
    Configuration menu
    Copy the full SHA
    7ae6194 View commit details
    Browse the repository at this point in the history
Loading