-{section: Old ClassAds ClassAd::Assign() [995 calls]} -{subsection: ClassAd::Assign(const char*&, int) [151 calls]:} -{code} -AvailStats.cpp: 113, 118, 120 -ResMgr.cpp: 1613, 1615 -Resource.cpp: 604, 1691 -amazonjob.cpp: 785 -basejob.cpp: 188, 210, 254, 334, 338, 372, 413, 445, 619, 700, 799, 934 -classad_helpers.cpp: 235, 236, 245, 248, 251, 252, 253, 254, 255, 256, 257, 258, 262, 263, 264, 270, 271, 273, 278, 282, 303, 304 -classad_visa.cpp: 64 -collector.cpp: 1435 -condorjob.cpp: 610, 1427, 1429, 1432, 1433, 1439, 1440, 1441, 1442, 1443, 1444, 1445, 1446, 1449 -creamjob.cpp: 1139 -daemon_core.cpp: 9560 -dedicated_scheduler.cpp: 692, 693, 694, 1033, 3403, 3408, 3410, 3412, 3447, 3449, 3451 -give_interactive.cpp: 267, 273, 274, 305, 306, 307, 308, 309, 310, 311 -globusjob.cpp: 948, 2127 -gridftpmanager.cpp: 557, 569 -gt4job.cpp: 1048, 1115, 1173, 1213, 1427 -lease_manager.cpp: 566 -master.cpp: 933 -matchmaker.cpp: 1705 -nordugridjob.cpp: 754 -schedd.cpp: 4324, 4950, 5000, 5110, 5830, 10504, 11488 -schedd_td.cpp: 92, 132, 168, 188, 316, 336, 354, 500, 558, 683, 694, 697, 1020, 1031, 1034 -shadow_common.cpp: 556, 662 -soap_scheddStub.cpp: 1574, 1575 -submit_job.cpp: 430, 435 -td_init.cpp: 451, 459 -td_read_files.cpp: 111, 130, 144, 159, 160, 259, 280, 301, 354, 370, 378 -td_write_files.cpp: 112, 131, 145, 160, 274, 295, 316, 372, 381, 390 -tdman.cpp: 932, 949, 966, 977 -vm_proc.cpp: 1193 -vmuniverse_mgr.cpp: 304 -{endcode} -{subsection: ClassAd::Assign(const char*&, int&) [129 calls]:} -{code} -ResAttributes.cpp: 270, 272, 274, 279, 282, 294, 307, 309, 500, 502, 504, 516, 521, 527, 529 -ResMgr.cpp: 1939 -ResState.cpp: 53, 57, 1086 -Resource.cpp: 1646, 1648, 1717, 1719, 1721 -autocluster.cpp: 272 -basejob.cpp: 187, 209, 253, 337, 341, 342, 354, 371, 395, 400, 437, 459, 556, 591, 618 -baseresource.cpp: 322, 342, 343 -baseshadow.cpp: 439, 440, 617, 1156 -classad_helpers.cpp: 232 -cod_tool.cpp: 294 -condor_event.cpp: 1341, 1342, 1341, 1342 -condor_secman.cpp: 502, 1385, 1389, 2822 -condorjob.cpp: 1431, 1448, 1487, 1580, 1581 -creamjob.cpp: 1329 -dc_schedd.cpp: 590, 652 -dc_transferd.cpp: 169, 342 -file_transfer.cpp: 1967, 1969, 1970, 2460, 2462, 2514, 2518, 2519 -globusjob.cpp: 1851, 2091, 2166, 2370, 2436, 2459, 2571, 2681, 2693 -gridftpmanager.cpp: 697, 698 -gt4job.cpp: 1414 -hibernation_manager.cpp: 246 -jic_local_schedd.cpp: 137, 138 -nordugridjob.cpp: 583, 586 -offline_plugin.cpp: 377 -parallelshadow.cpp: 735 -qmgmt.cpp: 3574 -remoteresource.cpp: 966, 971, 992, 997, 1219, 1235, 1239 -schedd.cpp: 5086, 11652, 11655, 11661, 11663, 11676, 11678, 12852 -schedd_client.cpp: 879, 880 -shadow.cpp: 342 -shadow_common.cpp: 570, 593, 676 -soap_scheddStub.cpp: 1564, 1565 -submit_job.cpp: 428, 429 -td_read_files.cpp: 355, 363 -td_write_files.cpp: 382, 391 -transfer_queue.cpp: 65 -vmuniverse_mgr.cpp: 189, 191, 301, 319 -{endcode} -{subsection: ClassAd::Assign(const char*&, const char*) [93 calls]:} -{code} -DBMSManager.cpp: 88 -ResAttributes.cpp: 229 -Resource.cpp: 1612, 1767 -Scheduler.cpp: 57 -amazonjob.cpp: 331 -basejob.cpp: 441, 479 -baseresource.cpp: 308 -ccb_listener.cpp: 62, 63, 69 -ccb_server.cpp: 246, 248, 468, 469, 471 -classad_command_util.cpp: 65 -classad_helpers.cpp: 307, 310 -collector.cpp: 1432 -command.cpp: 2032 -condor_arglist.cpp: 572, 582 -condor_secman.cpp: 488, 1376, 1889 -condorjob.cpp: 304, 1480 -condorresource.cpp: 213 -creamjob.cpp: 419, 571 -daemon_core.cpp: 4046, 4338 -dc_schedd.cpp: 591, 653, 655 -dc_startd.cpp: 403, 422, 442, 465, 469, 497, 501, 524, 550 -dedicated_scheduler.cpp: 3388 -env.cpp: 136, 177 -file_transfer.cpp: 2521 -globusjob.cpp: 837 -gridftpmanager.cpp: 564, 566 -gt4job.cpp: 473, 723 -infnbatchjob.cpp: 219, 853, 862 -java_proc.cpp: 111, 382, 387, 392 -jic_local.cpp: 368 -jic_shadow.cpp: 825, 1686 -lease_manager.cpp: 557 -master.cpp: 930 -network_adapter.cpp: 101, 102 -nordugridjob.cpp: 259 -qmgmt.cpp: 346, 385, 394, 424, 447, 474, 829, 840, 854 -remoteresource.cpp: 975, 979, 983, 1002, 1251, 1254 -schedd.cpp: 806, 11606 -shadow_common.cpp: 568, 613, 649, 674 -submit.cpp: 1598 -unicorejob.cpp: 253 -{endcode} -{subsection: ClassAd::Assign(const char[], int&) [93 calls]:} -{code} -amazonjob.cpp: 323, 386 -baseresource.cpp: 314, 315 -condor_event.cpp: 485, 486, 487, 1185, 1213, 1338, 1465, 1491, 1587, 1715, 2087, 2283, 2477, 2582, 2833, 2857, 2967, 3044, 3200, 3357, 485, 486, 487, 1185, 1213, 1338, 1465, 1491, 1587, 1715, 2087, 2283, 2477, 2582, 2833, 2857, 2967, 3044, 3200, 3357 -file_transfer_db.cpp: 189, 193, 195, 196, 198, 199, 201 -globusresource.cpp: 236, 239 -lease_manager.cpp: 583, 584, 585, 586 -partition.cpp: 98 -schedd.cpp: 11609, 11610, 11611, 11612, 11613, 11614, 11615, 11616, 11617, 11618, 11619, 11620, 11621, 11622, 11623, 11624, 11625, 11626, 11627, 11631, 11632, 11633, 11708, 11709, 11710, 11711 -self_monitor.cpp: 112, 115, 116 -workload.cpp: 83, 97, 112, 126, 141, 155 -{endcode} -{subsection: ClassAd::Assign(const char*&, char*&) [76 calls]:} -{code} -ResAttributes.cpp: 231, 233, 235, 237, 241 -Resource.cpp: 1614, 1768 -amazonresource.cpp: 124, 125 -basejob.cpp: 346 -baseresource.cpp: 310, 312 -claim.cpp: 209 -collector.cpp: 1427 -condor_secman.cpp: 430, 459, 476, 478, 480, 482, 492, 526, 2788 -condorjob.cpp: 1503, 1509, 1511 -condorresource.cpp: 215 -creamjob.cpp: 601, 633, 730 -creamresource.cpp: 304 -daemon_core.cpp: 4239, 4335 -dc_schedd.cpp: 614 -dc_startd.cpp: 405, 424, 444, 467, 499, 526 -dedicated_scheduler.cpp: 689, 690, 691, 1027, 3396 -give_interactive.cpp: 280 -globusresource.cpp: 235 -gridftpmanager.cpp: 559, 561, 562, 660 -gt4job.cpp: 752 -gt4resource.cpp: 253 -jic_local.cpp: 361, 372 -jic_shadow.cpp: 1198, 1209 -master.cpp: 919, 925 -nordugridresource.cpp: 120 -power.cpp: 243, 244 -pseudo_ops.cpp: 338 -qmgmt.cpp: 2809 -remoteresource.cpp: 1243, 1246 -schedd.cpp: 4939, 10498, 10511, 10513, 10520, 11641, 11647 -shadow_common.cpp: 620 -starter_mgr.cpp: 179 -submit.cpp: 1597 -{endcode} -{subsection: ClassAd::Assign(const char*&, const char[]) [75 calls]:} -{code} -amazonjob.cpp: 411 -basejob.cpp: 621 -classad_helpers.cpp: 260, 276, 280, 284, 285, 286, 287, 308, 321 -condor_secman.cpp: 484, 533, 537, 545, 1252, 1665, 2783, 2785 -condorjob.cpp: 396, 405, 1428 -creamjob.cpp: 515 -env.cpp: 195 -globusjob.cpp: 945, 968, 985, 994 -gridftpmanager.cpp: 558 -gt4job.cpp: 567, 584, 2209, 2223, 2242 -infnbatchjob.cpp: 284, 294, 348 -jic_shadow.cpp: 1774 -nordugridjob.cpp: 332, 341 -qmgmt.cpp: 464 -schedd_td.cpp: 93, 133, 169, 190, 296, 318, 338, 356 -submit_job.cpp: 436 -td_init.cpp: 462 -td_read_files.cpp: 112, 132, 146, 261, 282, 356, 368, 369, 374 -td_write_files.cpp: 113, 133, 147, 276, 297, 369, 379, 380, 387 -tdman.cpp: 934, 951, 968 -unicorejob.cpp: 318, 331, 407 -{endcode} -{subsection: ClassAd::Assign(const char*&, bool) [68 calls]:} -{code} -ResAttributes.cpp: 239 -ResMgr.cpp: 177 -Resource.cpp: 602, 1693 -amazonjob.cpp: 784 -basejob.cpp: 447, 463, 798, 812, 872 -classad_helpers.cpp: 249, 266, 267, 268, 274, 312, 314, 315, 316, 318, 319, 323 -command.cpp: 90 -condor_secman.cpp: 2789 -condorjob.cpp: 609, 949, 1447 -creamjob.cpp: 1138, 1328 -daemon_core.cpp: 4326 -dc_schedd.cpp: 592, 654 -dedicated_scheduler.cpp: 688, 1070, 3394 -give_interactive.cpp: 258, 264 -globusjob.cpp: 2126 -gridftpmanager.cpp: 670 -gt4job.cpp: 1212, 1452 -hibernation_manager.cpp: 250 -network_adapter.cpp: 103, 104, 105 -nordugridjob.cpp: 582, 585, 753 -remoteresource.cpp: 988, 1006 -schedd.cpp: 4329, 4963, 5109, 10509, 10523, 10527 -shadow_common.cpp: 555, 559, 588, 589, 592, 604, 615, 661, 665 -vmuniverse_mgr.cpp: 292, 296, 547 -{endcode} -{subsection: ClassAd::Assign(const char*&, const char*&) [46 calls]:} -{code} -amazonjob.cpp: 1139 -basejob.cpp: 270, 283, 294, 304, 340, 374 -baseshadow.cpp: 438 -ccb_client.cpp: 485 -ccb_listener.cpp: 327, 328, 331, 430 -ccb_server.cpp: 504 -classad_command_util.cpp: 67 -classad_helpers.cpp: 228, 233 -classad_visa.cpp: 71, 90 -condor_secman.cpp: 1381, 2784, 2790 -condorjob.cpp: 1465, 1475 -daemon_core.cpp: 4307, 9568, 9571, 9575 -dc_collector.cpp: 315, 320 -dc_startd.cpp: 552, 554, 557 -dc_transfer_queue.cpp: 209, 210 -file_transfer.cpp: 661, 1972 -hibernation_manager.cpp: 247 -jic_local_schedd.cpp: 136, 165, 195 -lease_manager.cpp: 546, 552 -qmgmt.cpp: 487, 489 -transfer_queue.cpp: 67 -{endcode} -{subsection: ClassAd::Assign(const char*&, MyString&) [40 calls]:} -{code} -baseresource.cpp: 217 -ccb_client.cpp: 145, 146, 481, 482 -ccb_server.cpp: 475 -condor_secman.cpp: 2772 -dc_schedd.cpp: 515, 516 -dc_transferd.cpp: 168, 341 -matchmaker.cpp: 2821 -network_adapter.cpp: 108, 109 -qmgmt.cpp: 867 -schedd.cpp: 11742 -schedd_td.cpp: 229, 230, 295, 685, 695, 696, 698, 1022, 1032, 1033, 1035 -td_init.cpp: 482 -td_read_files.cpp: 347, 358, 377 -td_write_files.cpp: 362, 371, 389 -vm_proc.cpp: 1212, 1216 -vmuniverse_mgr.cpp: 183, 186, 354, 357 -{endcode} -{subsection: ClassAd::Assign(const char[], char[]) [33 calls]:} -{code} -condor_event.cpp: 1187, 1214, 1329, 1332, 1486, 1589, 1717, 2093, 2284, 2470, 2835, 2858, 2968, 3045, 3201, 3358, 1187, 1214, 1329, 1332, 1486, 1589, 1717, 2093, 2284, 2470, 2835, 2858, 2968, 3045, 3201, 3358 -file_transfer_db.cpp: 188 -{endcode} -{subsection: ClassAd::Assign(const char[], ULogEventNumber) [26 calls]:} -{code} -condor_event.cpp: 1186, 1212, 1588, 1714, 2088, 2282, 2583, 2834, 2856, 2966, 3043, 3199, 3356, 1186, 1212, 1588, 1714, 2088, 2282, 2583, 2834, 2856, 2966, 3043, 3199, 3356 -{endcode} -{subsection: ClassAd::Assign(const char*&, char*) [15 calls]:} -{code} -DBMSManager.cpp: 87 -ResState.cpp: 51, 55 -Resource.cpp: 1782, 1785, 1787 -basejob.cpp: 412 -baseresource.cpp: 311 -ccb_client.cpp: 149 -classad_visa.cpp: 83 -collector.cpp: 1429 -daemon_core.cpp: 9563 -dedicated_scheduler.cpp: 3398, 3444 -lease_manager.cpp: 542 -{endcode} -{subsection: ClassAd::Assign(const char*&, float) [14 calls]:} -{code} -classad_helpers.cpp: 238, 239, 240, 241, 242 -condorjob.cpp: 1434, 1435, 1436, 1437, 1438 -parallelshadow.cpp: 741, 745 -vm_proc.cpp: 1203, 1204 -{endcode} -{subsection: ClassAd::Assign(const char[], float&) [13 calls]:} -{code} -condor_event.cpp: 2094, 2095, 2471, 2472, 2836, 2838, 2094, 2095, 2471, 2472, 2836, 2838 -self_monitor.cpp: 113 -{endcode} -{subsection: ClassAd::Assign(const char*&, float&) [13 calls]:} -{code} -basejob.cpp: 411, 900, 909 -matchmaker.cpp: 3112, 3119 -nordugridjob.cpp: 588, 589, 590 -remoteresource.cpp: 956, 961 -schedd.cpp: 11148 -vm_proc.cpp: 1191, 1192 -{endcode} -{subsection: ClassAd::Assign(const char[], MyString&) [13 calls]:} -{code} -file_transfer_db.cpp: 185, 187, 190, 191, 192, 194, 200 -partition.cpp: 69, 83, 149, 212 -vm_type.cpp: 172 -workload.cpp: 69 -{endcode} -{subsection: ClassAd::Assign(const char[], char*&) [11 calls]:} -{code} -condor_event.cpp: 478, 1335, 478, 1335 -file_transfer_db.cpp: 197 -infnbatchjob.cpp: 834 -schedd.cpp: 11607, 11628, 11629, 11630, 11712 -{endcode} -{subsection: ClassAd::Assign(const char[], int) [11 calls]:} -{code} -baseresource.cpp: 313, 316, 317, 318, 319 -globusresource.cpp: 237, 238, 240, 241 -schedd.cpp: 11608 -self_monitor.cpp: 117 -{endcode} -{subsection: ClassAd::Assign(const char[], const char[]) [8 calls]:} -{code} -partition.cpp: 117, 120, 123, 126, 129, 188, 191, 194 -{endcode} -{subsection: ClassAd::Assign(const char[], const char*) [7 calls]:} -{code} -baseresource.cpp: 309 -lease_manager.cpp: 559, 562 -user_log.cpp: 994, 994 -vm_type.cpp: 562 -xen_type.cpp: 1655 -{endcode} -{subsection: ClassAd::Assign(const char*&, bool&) [6 calls]:} -{code} -baseresource.cpp: 722 -ccb_listener.cpp: 428 -ccb_server.cpp: 503 -dc_transfer_queue.cpp: 208 -file_transfer.cpp: 2517 -vmuniverse_mgr.cpp: 321 -{endcode} -{subsection: ClassAd::Assign(const char*&, const int&) [5 calls]:} -{code} -ccb_listener.cpp: 58 -ccb_server.cpp: 247, 467 -globusjob.cpp: 947 -gt4job.cpp: 1453 -{endcode} -{subsection: ClassAd::Assign(const char[], const char*&) [5 calls]:} -{code} -collector_engine.cpp: 446 -condor_event.cpp: 482, 482 -matchmaker.cpp: 3478, 3518 -{endcode} -{subsection: ClassAd::Assign(const char*&, double) [5 calls]:} -{code} -Resource.cpp: 1625 -give_interactive.cpp: 301, 302, 303, 304 -{endcode} -{subsection: ClassAd::Assign(const char*, const char*) [4 calls]:} -{code} -classad_command_util.cpp: 36, 38 -soap_scheddStub.cpp: 1561, 1562 -{endcode} -{subsection: ClassAd::Assign(const char[], ULogEventNumber&) [4 calls]:} -{code} -user_log.cpp: 993, 997, 993, 997 -{endcode} -{subsection: ClassAd::Assign(const char*&, char[]) [3 calls]:} -{code} -env.cpp: 165 -gridftpmanager.cpp: 598 -power.cpp: 245 -{endcode} -{subsection: ClassAd::Assign(char*&, bool) [2 calls]:} -{code} -ClassAdReevaluator.cpp: 151 -matchmaker.cpp: 781 -{endcode} -{subsection: ClassAd::Assign(char* const, float&) [2 calls]:} -{code} -user_log.cpp: 975, 975 -{endcode} -{subsection: ClassAd::Assign(const char*&, MyString) [2 calls]:} -{code} -ccb_client.cpp: 148, 484 -{endcode} -{subsection: ClassAd::Assign(const char*&, unsigned int&) [2 calls]:} -{code} -dc_collector.cpp: 302, 306 -{endcode} -{subsection: ClassAd::Assign(const char*&, time_t) [2 calls]:} -{code} -dc_collector.cpp: 295, 298 -{endcode} -{subsection: ClassAd::Assign(char* const, int&) [2 calls]:} -{code} -user_log.cpp: 971, 971 -{endcode} -{subsection: ClassAd::Assign(char* const, char*&) [2 calls]:} -{code} -user_log.cpp: 979, 979 -{endcode} -{subsection: ClassAd::Assign(const char*&, FTPMode) [2 calls]:} -{code} -dc_schedd.cpp: 624, 662 -{endcode} -{subsection: ClassAd::Assign(const char*, float&) [1 calls]:} -{code} -matchmaker.cpp: 3144 -{endcode} -{subsection: ClassAd::Assign(const char*&, double&) [1 calls]:} -{code} -give_interactive.cpp: 259 -{endcode} -{subsection: ClassAd::Assign(const char[], AdTypes&) [1 calls]:} -{code} -lease_manager.cpp: 560 -{endcode} -{subsection: ClassAd::Assign(const char*&, JobAction&) [1 calls]:} -{code} -schedd.cpp: 4320 -{endcode} -{subsection: ClassAd::Assign(char*&, char*&) [1 calls]:} -{code} -ClassAdReevaluator.cpp: 82 -{endcode} -{subsection: ClassAd::Assign(char*&, int&) [1 calls]:} -{code} -ClassAdReevaluator.cpp: 107 -{endcode} -{subsection: ClassAd::Assign(const char[], bool) [1 calls]:} -{code} -soap_scheddStub.cpp: 1577 -{endcode} -{subsection: ClassAd::Assign(char*&, float&) [1 calls]:} -{code} -ClassAdReevaluator.cpp: 129 -{endcode} -{subsection: ClassAd::Assign(const char*&, __pid_t) [1 calls]:} -{code} -classad_visa.cpp: 77 -{endcode} -{subsection: ClassAd::Assign(const char*, char*&) [1 calls]:} -{code} -collector.cpp: 1422 -{endcode} -{subsection: ClassAd::Assign(const char*&, unsigned char*&) [1 calls]:} -{code} -condor_secman.cpp: 1343 -{endcode} -{subsection: ClassAd::Assign(const char[], float) [1 calls]:} -{code} -self_monitor.cpp: 114 -{endcode} +This method has six variants: +*:int Assign(char const *variable, MyString &value) +*:int Assign(char const *variable,char const *value) +*:int Assign(char const *variable,int value) +*:int Assign(char const *variable,float value) +*:int Assign(char const *variable,double value) +*:int Assign(char const *variable,bool value) + +In new {quote:ClassAds}, InsertAttr() is a direct equivalent set of +methods. The only differences are the return type (int vs. bool) and new +{quote:ClassAds} don't have float or MyString variants. + +A simple search-and-replace or set of pass-through functions will work. +In the former case, some call sites may need to be fixed up because of +the change in return type. For the MyString variant, we can try adding a +cast operator to std::string or const char* in MyString. + +Developer time: minutes, some student time for search-and-replace fixup