-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 a float variant. - -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. - -Developer time: minutes, some student time for search-and-replace fixup +{section: Old ClassAds ClassAd::Assign() [995 calls]} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: ClassAd::Assign(const char[], const char[]) [8 calls]:} +{code} + partition.cpp: 117, 120, 123, 126, 129, 188, 191, 194 +{endcode} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: 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} +{subsubsection: ClassAd::Assign(const char[], const char*&) [5 calls]:} +{code} + collector_engine.cpp: 446 + condor_event.cpp: 482, 482 + matchmaker.cpp: 3478, 3518 +{endcode} +{subsubsection: ClassAd::Assign(const char*&, double) [5 calls]:} +{code} + Resource.cpp: 1625 + give_interactive.cpp: 301, 302, 303, 304 +{endcode} +{subsubsection: ClassAd::Assign(const char*, const char*) [4 calls]:} +{code} + classad_command_util.cpp: 36, 38 + soap_scheddStub.cpp: 1561, 1562 +{endcode} +{subsubsection: ClassAd::Assign(const char[], ULogEventNumber&) [4 calls]:} +{code} + user_log.cpp: 993, 997, 993, 997 +{endcode} +{subsubsection: ClassAd::Assign(const char*&, char[]) [3 calls]:} +{code} + env.cpp: 165 + gridftpmanager.cpp: 598 + power.cpp: 245 +{endcode} +{subsubsection: ClassAd::Assign(char*&, bool) [2 calls]:} +{code} + ClassAdReevaluator.cpp: 151 + matchmaker.cpp: 781 +{endcode} +{subsubsection: ClassAd::Assign(char* const, float&) [2 calls]:} +{code} + user_log.cpp: 975, 975 +{endcode} +{subsubsection: ClassAd::Assign(const char*&, MyString) [2 calls]:} +{code} + ccb_client.cpp: 148, 484 +{endcode} +{subsubsection: ClassAd::Assign(const char*&, unsigned int&) [2 calls]:} +{code} + dc_collector.cpp: 302, 306 +{endcode} +{subsubsection: ClassAd::Assign(const char*&, time_t) [2 calls]:} +{code} + dc_collector.cpp: 295, 298 +{endcode} +{subsubsection: ClassAd::Assign(char* const, int&) [2 calls]:} +{code} + user_log.cpp: 971, 971 +{endcode} +{subsubsection: ClassAd::Assign(char* const, char*&) [2 calls]:} +{code} + user_log.cpp: 979, 979 +{endcode} +{subsubsection: ClassAd::Assign(const char*&, FTPMode) [2 calls]:} +{code} + dc_schedd.cpp: 624, 662 +{endcode} +{subsubsection: ClassAd::Assign(const char*, float&) [1 calls]:} +{code} + matchmaker.cpp: 3144 +{endcode} +{subsubsection: ClassAd::Assign(const char*&, double&) [1 calls]:} +{code} + give_interactive.cpp: 259 +{endcode} +{subsubsection: ClassAd::Assign(const char[], AdTypes&) [1 calls]:} +{code} + lease_manager.cpp: 560 +{endcode} +{subsubsection: ClassAd::Assign(const char*&, JobAction&) [1 calls]:} +{code} + schedd.cpp: 4320 +{endcode} +{subsubsection: ClassAd::Assign(char*&, char*&) [1 calls]:} +{code} + ClassAdReevaluator.cpp: 82 +{endcode} +{subsubsection: ClassAd::Assign(char*&, int&) [1 calls]:} +{code} + ClassAdReevaluator.cpp: 107 +{endcode} +{subsubsection: ClassAd::Assign(const char[], bool) [1 calls]:} +{code} + soap_scheddStub.cpp: 1577 +{endcode} +{subsubsection: ClassAd::Assign(char*&, float&) [1 calls]:} +{code} + ClassAdReevaluator.cpp: 129 +{endcode} +{subsubsection: ClassAd::Assign(const char*&, __pid_t) [1 calls]:} +{code} + classad_visa.cpp: 77 +{endcode} +{subsubsection: ClassAd::Assign(const char*, char*&) [1 calls]:} +{code} + collector.cpp: 1422 +{endcode} +{subsubsection: ClassAd::Assign(const char*&, unsigned char*&) [1 calls]:} +{code} + condor_secman.cpp: 1343 +{endcode} +{subsubsection: ClassAd::Assign(const char[], float) [1 calls]:} +{code} + self_monitor.cpp: 114 +{endcode}