Merge branch 'ondrej/cleanup-double-createinctx-constructors' into 'main'

Cleanup the isc_<*>mgr_createinc() constructors

See merge request isc-projects/bind9!4910

bin/delv/delv.c

@@ -1759,9 +1759,9 @@ main(int argc, char *argv[]) {
 	isc_mem_create(&mctx);
 
 	CHECK(isc_appctx_create(mctx, &actx));
-	CHECK(isc_taskmgr_createinctx(mctx, 1, 0, &taskmgr));
-	CHECK(isc_socketmgr_createinctx(mctx, &socketmgr));
-	CHECK(isc_timermgr_createinctx(mctx, &timermgr));
+	CHECK(isc_taskmgr_create(mctx, 1, 0, NULL, &taskmgr));
+	CHECK(isc_socketmgr_create(mctx, &socketmgr));
+	CHECK(isc_timermgr_create(mctx, &timermgr));
 
 	parse_args(argc, argv);
 

bin/tests/system/resolve.c

@@ -371,15 +371,15 @@ main(int argc, char *argv[]) {
 	if (result != ISC_R_SUCCESS) {
 		goto cleanup;
 	}
-	result = isc_taskmgr_createinctx(mctx, 1, 0, &taskmgr);
+	result = isc_taskmgr_create(mctx, 1, 0, NULL, &taskmgr);
 	if (result != ISC_R_SUCCESS) {
 		goto cleanup;
 	}
-	result = isc_socketmgr_createinctx(mctx, &socketmgr);
+	result = isc_socketmgr_create(mctx, &socketmgr);
 	if (result != ISC_R_SUCCESS) {
 		goto cleanup;
 	}
-	result = isc_timermgr_createinctx(mctx, &timermgr);
+	result = isc_timermgr_create(mctx, &timermgr);
 	if (result != ISC_R_SUCCESS) {
 		goto cleanup;
 	}

lib/isc/include/isc/socket.h

@@ -715,9 +715,6 @@ isc_socket_sendto2(isc_socket_t *sock, isc_region_t *region, isc_task_t *task,
  */
 /*@}*/
 
-isc_result_t
-isc_socketmgr_createinctx(isc_mem_t *mctx, isc_socketmgr_t **managerp);
-
 isc_result_t
 isc_socketmgr_create(isc_mem_t *mctx, isc_socketmgr_t **managerp);
 
@@ -729,8 +726,6 @@ isc_socketmgr_create2(isc_mem_t *mctx, isc_socketmgr_t **managerp,
  * maximum number of sockets that the created manager should handle.
  * isc_socketmgr_create() is equivalent of isc_socketmgr_create2() with
  * "maxsocks" being zero.
- * isc_socketmgr_createinctx() also associates the new manager with the
- * specified application context.
  *
  * Notes:
  *

lib/isc/include/isc/task.h

@@ -640,16 +640,12 @@ isc_task_privilege(isc_task_t *task);
 ***** Task Manager.
 *****/
 
-isc_result_t
-isc_taskmgr_createinctx(isc_mem_t *mctx, unsigned int workers,
-			unsigned int default_quantum, isc_taskmgr_t **managerp);
 isc_result_t
 isc_taskmgr_create(isc_mem_t *mctx, unsigned int workers,
 		   unsigned int default_quantum, isc_nm_t *nm,
 		   isc_taskmgr_t **managerp);
 /*%<
- * Create a new task manager.  isc_taskmgr_createinctx() also associates
- * the new manager with the specified application context.
+ * Create a new task manager.
  *
  * Notes:
  *
@@ -674,8 +670,6 @@ isc_taskmgr_create(isc_mem_t *mctx, unsigned int workers,
  *
  *\li	managerp != NULL && *managerp == NULL
  *
- *\li	'actx' is a valid application context (for createinctx()).
- *
  * Ensures:
  *
  *\li	On success, '*managerp' will be attached to the newly created task

lib/isc/include/isc/timer.h

@@ -306,14 +306,10 @@ isc_timer_gettype(isc_timer_t *timer);
  *\li	'timer' to be a valid timer.
  */
 
-isc_result_t
-isc_timermgr_createinctx(isc_mem_t *mctx, isc_timermgr_t **managerp);
-
 isc_result_t
 isc_timermgr_create(isc_mem_t *mctx, isc_timermgr_t **managerp);
 /*%<
- * Create a timer manager.  isc_timermgr_createinctx() also associates
- * the new manager with the specified application context.
+ * Create a timer manager.
  *
  * Notes:
  *
@@ -325,8 +321,6 @@ isc_timermgr_create(isc_mem_t *mctx, isc_timermgr_t **managerp);
  *
  *\li	'managerp' points to a NULL isc_timermgr_t.
  *
- *\li	'actx' is a valid application context (for createinctx()).
- *
  * Ensures:
  *
  *\li	'*managerp' is a valid isc_timermgr_t.

lib/isc/task.c

@@ -2015,15 +2015,3 @@ error:
 	return (result);
 }
 #endif /* ifdef HAVE_JSON_C */
-
-isc_result_t
-isc_taskmgr_createinctx(isc_mem_t *mctx, unsigned int workers,
-			unsigned int default_quantum,
-			isc_taskmgr_t **managerp) {
-	isc_result_t result;
-
-	result = isc_taskmgr_create(mctx, workers, default_quantum, NULL,
-				    managerp);
-
-	return (result);
-}

lib/isc/timer.c

@@ -770,12 +770,3 @@ isc_timermgr_destroy(isc_timermgr_t **managerp) {
 
 	*managerp = NULL;
 }
-
-isc_result_t
-isc_timermgr_createinctx(isc_mem_t *mctx, isc_timermgr_t **managerp) {
-	isc_result_t result;
-
-	result = isc_timermgr_create(mctx, managerp);
-
-	return (result);
-}

lib/isc/unix/socket.c

@@ -5554,12 +5554,3 @@ error:
 	return (result);
 }
 #endif /* HAVE_JSON_C */
-
-isc_result_t
-isc_socketmgr_createinctx(isc_mem_t *mctx, isc_socketmgr_t **managerp) {
-	isc_result_t result;
-
-	result = isc_socketmgr_create(mctx, managerp);
-
-	return (result);
-}

lib/isc/win32/libisc.def.in

@@ -591,7 +591,6 @@ isc_sockaddr_setport
 isc_sockaddr_totext
 isc_sockaddr_v6fromin
 isc_socket_socketevent
-isc_socketmgr_createinctx
 isc_socketmgr_maxudp
 @IF NOTYET
 isc_socketmgr_renderjson
@@ -653,7 +652,6 @@ isc_task_shutdown
 isc_task_unpause
 isc_task_unsend
 isc_taskmgr_create
-isc_taskmgr_createinctx
 isc_taskmgr_destroy
 isc_taskmgr_excltask
 isc_taskmgr_mode
@@ -706,7 +704,6 @@ isc_timer_gettype
 isc_timer_reset
 isc_timer_touch
 isc_timermgr_create
-isc_timermgr_createinctx
 isc_timermgr_destroy
 isc_timermgr_poke
 isc_tls_get_http2_alpn

lib/isc/win32/socket.c

@@ -3955,15 +3955,6 @@ error:
 }
 #endif /* HAVE_JSON_C */
 
-isc_result_t
-isc_socketmgr_createinctx(isc_mem_t *mctx, isc_socketmgr_t **managerp) {
-	isc_result_t result;
-
-	result = isc_socketmgr_create(mctx, managerp);
-
-	return (result);
-}
-
 void
 isc_socketmgr_maxudp(isc_socketmgr_t *manager, unsigned int maxudp) {
 	REQUIRE(VALID_MANAGER(manager));