Ticket #196 (closed enhancement: fixed)

Opened 4 years ago

Last modified 4 years ago

move send_two_return_one to board library

Reported by: gunnar Owned by: gnugo
Priority: normal Milestone: 3.7.12
Component: source Version:
Severity: normal Keywords:
Cc: patch: yes

Description

This patch moves the send_two_return_one() function from utils.c to board.c and implements it as a low level board function. This has two implications:

  1. It becomes faster. This is probably not critical but since it's called from the tactical reading it can't hurt.
  2. More importantly, the tactical reading sgf dumps are no longer cluttered by uninteresting trymoves from this function after a ko capture.

Attachments

gunnar_7_12.4.diff Download (4.9 KB) - added by gunnar 4 years ago.
Turn send_two_return_one() into a low level board function.
gunnar_7_12.4b.diff Download (5.0 KB) - added by gunnar 4 years ago.
Second attempt.

Regression Results

Attachment Rev. PASS FAIL Nodes Status
gunnar_7_12.4.diff Download 2401 2 +0.046% +0.065% -0.011% details
gunnar_7_12.4b.diff Download 2407 -0.0075% -0.00075% -0.00026% builds with warning(s) details

Change History

Changed 4 years ago by gunnar

Turn send_two_return_one() into a low level board function.

Changed 4 years ago by gunnar

Second attempt.

comment:1 Changed 4 years ago by gunnar

  • Status changed from new to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.