* [PATCH][GSOC2014]
@ 2014-03-10 19:04 TamerTas
0 siblings, 0 replies; only message in thread
From: TamerTas @ 2014-03-10 19:04 UTC (permalink / raw)
To: git; +Cc: TamerTas
I inspected branch.c:install_branch_config() and since
the conditionals cover all the possibilities, I decided
that making the code table-driven would be much cleaner
and shorter. I've ran the tests and they all passed.
Reimplementation using "git am" also didn't have any problems.
Please let me know if there is any problems that I missed.
I would also love to learn if you think that the code was better
before and why it was better.
TamerTas (1):
changed logical chain in branch.c to lookup tables
branch.c | 31 ++++++++-----------------------
1 file changed, 8 insertions(+), 23 deletions(-)
--
1.7.9.5
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2014-03-10 19:09 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-03-10 19:04 [PATCH][GSOC2014] TamerTas
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).