Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Version 3 proposal: Remove use_zero_index_for_rank_option #27

Closed
wants to merge 3 commits into from
Closed

Version 3 proposal: Remove use_zero_index_for_rank_option #27

wants to merge 3 commits into from

Commits on Sep 16, 2012

  1. Version 3 proposal: Remove use_zero_index_for_rank_option 

    Is this ever really used in practice? My guess is no. Although it
could be left in as it's not necessarily a lot of code, but it
does remove some conditional paths.
    David Czarnecki committed Sep 16, 2012
    Configuration menu
    Copy the full SHA
    3a85706 View commit details
    Browse the repository at this point in the history

  2. Version 3 proposal: Remove use_zero_index_for_rank_option 

    Is this ever really used in practice? My guess is no. Although it
could be left in as it's not necessarily a lot of code, but it
does remove some conditional paths.
    David Czarnecki committed Sep 16, 2012
    Configuration menu
    Copy the full SHA
    e621eee View commit details
    Browse the repository at this point in the history

  3. Rebase from master

    David Czarnecki committed Sep 16, 2012
    Configuration menu
    Copy the full SHA
    b05e1ad View commit details
    Browse the repository at this point in the history