forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commita79b698
committed
Here's a version of my suggested diffs transplanted to 7.1 beta 5. I'mstill looking at the best way to integrate Tom Vijlbrief's fixes(insofar as they're still needed); would 7.2 be a suitable time forincompatible API changes?JeroenChanges:(*) Introduced bool, true, false (replacing some int, 1, 0)(*) Made some member functions const(*) Documented GetIsNull()(*) Marked DisplayTuples() and PrintTuples() as obsolescent; fixed possible portability problem (assumed that NULL pointer equals all-zero bit pattern)(*) PrintTuples(): renamed width parameter to fillAlign to conform with other usage; fixed memory leak and compile issue w.r.t. field separator (should also slightly improve performance)(*) Fixed some minor compilation issues(*) Moved "using namespace std;" out of headers, where they didn't belong; used new (temporary) preprocessor macro PGSTD to do this(*) Made ToString() static, removed unneeded memset(), made buffer size adapt to sizeof(int)(*) Made some constructors explicit(*) Changed some const std::string & parameters to plain std::string(*) Marked PgCursor::Cursor(std::string) as obsolescent (setter with same name as getter--bad style)(*) Renamed some paramaters previously named "string"(*) Introduced size_type typedef for number of tuples in result set(*) PgTransaction now supports re-opening after closing, and aborts if not explicitly committed prior to destructionJ. T. Vermeulen
1 parentc8db55f commita79b698
File tree
18 files changed
+338
-146
lines changed- doc/src/sgml
- ref
- src
- bin/psql
- interfaces/libpq++
- examples
18 files changed
+338
-146
lines changedLines changed: 46 additions & 24 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1 | 1 |
| |
2 |
| - | |
| 2 | + | |
3 | 3 |
| |
4 | 4 |
| |
5 | 5 |
| |
| |||
227 | 227 |
| |
228 | 228 |
| |
229 | 229 |
| |
230 |
| - | |
| 230 | + | |
231 | 231 |
| |
232 |
| - | |
| 232 | + | |
233 | 233 |
| |
234 | 234 |
| |
235 | 235 |
| |
| |||
368 | 368 |
| |
369 | 369 |
| |
370 | 370 |
| |
371 |
| - | |
| 371 | + | |
372 | 372 |
| |
373 | 373 |
| |
374 | 374 |
| |
| |||
387 | 387 |
| |
388 | 388 |
| |
389 | 389 |
| |
390 |
| - | |
| 390 | + | |
391 | 391 |
| |
392 | 392 |
| |
393 | 393 |
| |
| |||
397 | 397 |
| |
398 | 398 |
| |
399 | 399 |
| |
400 |
| - | |
| 400 | + | |
401 | 401 |
| |
402 | 402 |
| |
403 | 403 |
| |
| |||
409 | 409 |
| |
410 | 410 |
| |
411 | 411 |
| |
412 |
| - | |
| 412 | + | |
413 | 413 |
| |
414 | 414 |
| |
415 | 415 |
| |
| |||
420 | 420 |
| |
421 | 421 |
| |
422 | 422 |
| |
423 |
| - | |
| 423 | + | |
424 | 424 |
| |
425 | 425 |
| |
426 | 426 |
| |
| |||
430 | 430 |
| |
431 | 431 |
| |
432 | 432 |
| |
433 |
| - | |
| 433 | + | |
434 | 434 |
| |
435 | 435 |
| |
436 | 436 |
| |
| |||
444 | 444 |
| |
445 | 445 |
| |
446 | 446 |
| |
447 |
| - | |
| 447 | + | |
448 | 448 |
| |
449 | 449 |
| |
450 | 450 |
| |
| |||
466 | 466 |
| |
467 | 467 |
| |
468 | 468 |
| |
469 |
| - | |
| 469 | + | |
470 | 470 |
| |
471 | 471 |
| |
472 | 472 |
| |
| |||
486 | 486 |
| |
487 | 487 |
| |
488 | 488 |
| |
489 |
| - | |
| 489 | + | |
490 | 490 |
| |
491 | 491 |
| |
492 | 492 |
| |
| |||
506 | 506 |
| |
507 | 507 |
| |
508 | 508 |
| |
509 |
| - | |
| 509 | + | |
510 | 510 |
| |
511 | 511 |
| |
512 | 512 |
| |
| |||
520 | 520 |
| |
521 | 521 |
| |
522 | 522 |
| |
523 |
| - | |
| 523 | + | |
524 | 524 |
| |
525 | 525 |
| |
526 | 526 |
| |
527 | 527 |
| |
528 | 528 |
| |
529 | 529 |
| |
530 | 530 |
| |
| 531 | + | |
| 532 | + | |
| 533 | + | |
| 534 | + | |
| 535 | + | |
| 536 | + | |
| 537 | + | |
| 538 | + | |
| 539 | + | |
| 540 | + | |
| 541 | + | |
| 542 | + | |
| 543 | + | |
| 544 | + | |
| 545 | + | |
| 546 | + | |
| 547 | + | |
| 548 | + | |
| 549 | + | |
| 550 | + | |
| 551 | + | |
| 552 | + | |
531 | 553 |
| |
532 | 554 |
| |
533 | 555 |
| |
534 |
| - | |
| 556 | + | |
535 | 557 |
| |
536 | 558 |
| |
537 |
| - | |
538 |
| - | |
| 559 | + | |
| 560 | + | |
539 | 561 |
| |
540 | 562 |
| |
541 | 563 |
| |
542 | 564 |
| |
543 | 565 |
| |
544 | 566 |
| |
545 |
| - | |
| 567 | + | |
546 | 568 |
| |
547 | 569 |
| |
548 |
| - | |
549 |
| - | |
| 570 | + | |
| 571 | + | |
550 | 572 |
| |
551 | 573 |
| |
552 | 574 |
| |
| |||
563 | 585 |
| |
564 | 586 |
| |
565 | 587 |
| |
566 |
| - | |
| 588 | + | |
567 | 589 |
| |
568 | 590 |
| |
569 | 591 |
| |
| |||
572 | 594 |
| |
573 | 595 |
| |
574 | 596 |
| |
575 |
| - | |
| 597 | + | |
576 | 598 |
| |
577 | 599 |
| |
578 | 600 |
| |
| |||
650 | 672 |
| |
651 | 673 |
| |
652 | 674 |
| |
653 |
| - | |
654 |
| - | |
| 675 | + | |
| 676 | + | |
655 | 677 |
| |
656 | 678 |
| |
657 | 679 |
| |
|
Lines changed: 11 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1 | 1 |
| |
2 |
| - | |
| 2 | + | |
3 | 3 |
| |
4 | 4 |
| |
5 | 5 |
| |
| |||
494 | 494 |
| |
495 | 495 |
| |
496 | 496 |
| |
| 497 | + | |
| 498 | + | |
| 499 | + | |
| 500 | + | |
| 501 | + | |
| 502 | + | |
| 503 | + | |
| 504 | + | |
| 505 | + | |
| 506 | + | |
497 | 507 |
| |
498 | 508 |
| |
499 | 509 |
| |
|
Lines changed: 4 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
3 | 3 |
| |
4 | 4 |
| |
5 | 5 |
| |
6 |
| - | |
| 6 | + | |
7 | 7 |
| |
8 | 8 |
| |
9 | 9 |
| |
| |||
357 | 357 |
| |
358 | 358 |
| |
359 | 359 |
| |
| 360 | + | |
| 361 | + | |
| 362 | + | |
360 | 363 |
| |
361 | 364 |
| |
362 | 365 |
| |
|
Lines changed: 98 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
3 | 3 |
| |
4 | 4 |
| |
5 | 5 |
| |
6 |
| - | |
| 6 | + | |
7 | 7 |
| |
8 | 8 |
| |
9 | 9 |
| |
| |||
890 | 890 |
| |
891 | 891 |
| |
892 | 892 |
| |
| 893 | + | |
| 894 | + | |
| 895 | + | |
| 896 | + | |
| 897 | + | |
| 898 | + | |
| 899 | + | |
| 900 | + | |
| 901 | + | |
| 902 | + | |
| 903 | + | |
| 904 | + | |
| 905 | + | |
| 906 | + | |
| 907 | + | |
| 908 | + | |
| 909 | + | |
| 910 | + | |
| 911 | + | |
| 912 | + | |
| 913 | + | |
| 914 | + | |
| 915 | + | |
| 916 | + | |
| 917 | + | |
| 918 | + | |
| 919 | + | |
| 920 | + | |
| 921 | + | |
| 922 | + | |
| 923 | + | |
| 924 | + | |
| 925 | + | |
| 926 | + | |
| 927 | + | |
| 928 | + | |
| 929 | + | |
| 930 | + | |
| 931 | + | |
| 932 | + | |
| 933 | + | |
| 934 | + | |
| 935 | + | |
| 936 | + | |
| 937 | + | |
| 938 | + | |
| 939 | + | |
| 940 | + | |
| 941 | + | |
| 942 | + | |
| 943 | + | |
| 944 | + | |
| 945 | + | |
| 946 | + | |
| 947 | + | |
| 948 | + | |
| 949 | + | |
| 950 | + | |
| 951 | + | |
| 952 | + | |
| 953 | + | |
| 954 | + | |
| 955 | + | |
| 956 | + | |
| 957 | + | |
| 958 | + | |
| 959 | + | |
| 960 | + | |
| 961 | + | |
| 962 | + | |
| 963 | + | |
| 964 | + | |
| 965 | + | |
| 966 | + | |
| 967 | + | |
| 968 | + | |
| 969 | + | |
| 970 | + | |
| 971 | + | |
| 972 | + | |
| 973 | + | |
| 974 | + | |
| 975 | + | |
| 976 | + | |
| 977 | + | |
| 978 | + | |
| 979 | + | |
| 980 | + | |
| 981 | + | |
| 982 | + | |
| 983 | + | |
| 984 | + | |
| 985 | + | |
| 986 | + | |
| 987 | + | |
| 988 | + | |
| 989 | + | |
893 | 990 |
| |
894 | 991 |
| |
895 | 992 |
| |
|
Lines changed: 4 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
3 | 3 |
| |
4 | 4 |
| |
5 | 5 |
| |
6 |
| - | |
| 6 | + | |
7 | 7 |
| |
8 | 8 |
| |
9 | 9 |
| |
| |||
22 | 22 |
| |
23 | 23 |
| |
24 | 24 |
| |
| 25 | + | |
| 26 | + | |
| 27 | + | |
25 | 28 |
| |
26 | 29 |
| |
27 | 30 |
| |
|
Lines changed: 2 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
22 | 22 |
| |
23 | 23 |
| |
24 | 24 |
| |
25 |
| - | |
| 25 | + | |
26 | 26 |
| |
27 | 27 |
| |
28 | 28 |
| |
29 | 29 |
| |
30 |
| - | |
| 30 | + | |
31 | 31 |
| |
32 | 32 |
| |
33 | 33 |
| |
|
0 commit comments
Comments
(0)