Fossil SCM
Fix comment typo, and add comment about possible simplification.
Commit
83ecb5b9453bfd6a01c376e70ea319cb4d8f4927
Parent
2e84fb3e544008c…
2 files changed
+1
+1
-1
| --- tools/cvs2fossil/lib/c2f_patopsort.tcl | ||
| +++ tools/cvs2fossil/lib/c2f_patopsort.tcl | ||
| @@ -107,10 +107,11 @@ | ||
| 107 | 107 | ## Internal methods |
| 108 | 108 | |
| 109 | 109 | proc Changesets {} { project::rev all } |
| 110 | 110 | |
| 111 | 111 | proc LoadSymbolChangesets {} { |
| 112 | + # Consider use of 'project::rev sym' here. | |
| 112 | 113 | set mysymchangesets [struct::list filter [project::rev all] [myproc IsBySymbol]] |
| 113 | 114 | return |
| 114 | 115 | } |
| 115 | 116 | |
| 116 | 117 | proc IsBySymbol {cset} { $cset bysymbol } |
| 117 | 118 |
| --- tools/cvs2fossil/lib/c2f_patopsort.tcl | |
| +++ tools/cvs2fossil/lib/c2f_patopsort.tcl | |
| @@ -107,10 +107,11 @@ | |
| 107 | ## Internal methods |
| 108 | |
| 109 | proc Changesets {} { project::rev all } |
| 110 | |
| 111 | proc LoadSymbolChangesets {} { |
| 112 | set mysymchangesets [struct::list filter [project::rev all] [myproc IsBySymbol]] |
| 113 | return |
| 114 | } |
| 115 | |
| 116 | proc IsBySymbol {cset} { $cset bysymbol } |
| 117 |
| --- tools/cvs2fossil/lib/c2f_patopsort.tcl | |
| +++ tools/cvs2fossil/lib/c2f_patopsort.tcl | |
| @@ -107,10 +107,11 @@ | |
| 107 | ## Internal methods |
| 108 | |
| 109 | proc Changesets {} { project::rev all } |
| 110 | |
| 111 | proc LoadSymbolChangesets {} { |
| 112 | # Consider use of 'project::rev sym' here. |
| 113 | set mysymchangesets [struct::list filter [project::rev all] [myproc IsBySymbol]] |
| 114 | return |
| 115 | } |
| 116 | |
| 117 | proc IsBySymbol {cset} { $cset bysymbol } |
| 118 |
| --- tools/cvs2fossil/lib/c2f_pbreakacycle.tcl | ||
| +++ tools/cvs2fossil/lib/c2f_pbreakacycle.tcl | ||
| @@ -337,11 +337,11 @@ | ||
| 337 | 337 | |
| 338 | 338 | # This works if and only if none of the symbol changesets are |
| 339 | 339 | # "backwards", hence our breaking of the backward changesets |
| 340 | 340 | # first, in the pre-hook. |
| 341 | 341 | |
| 342 | - # Note that tah changesets cannot be backward as they don't | |
| 342 | + # Note that tag changesets cannot be backward as they don't | |
| 343 | 343 | # have successors at all. |
| 344 | 344 | |
| 345 | 345 | # An interesting thing IMHO, is that after breaking the |
| 346 | 346 | # backward symbol changesets we should not have any circles |
| 347 | 347 | # any longer. Each circle which would still be present has to |
| 348 | 348 |
| --- tools/cvs2fossil/lib/c2f_pbreakacycle.tcl | |
| +++ tools/cvs2fossil/lib/c2f_pbreakacycle.tcl | |
| @@ -337,11 +337,11 @@ | |
| 337 | |
| 338 | # This works if and only if none of the symbol changesets are |
| 339 | # "backwards", hence our breaking of the backward changesets |
| 340 | # first, in the pre-hook. |
| 341 | |
| 342 | # Note that tah changesets cannot be backward as they don't |
| 343 | # have successors at all. |
| 344 | |
| 345 | # An interesting thing IMHO, is that after breaking the |
| 346 | # backward symbol changesets we should not have any circles |
| 347 | # any longer. Each circle which would still be present has to |
| 348 |
| --- tools/cvs2fossil/lib/c2f_pbreakacycle.tcl | |
| +++ tools/cvs2fossil/lib/c2f_pbreakacycle.tcl | |
| @@ -337,11 +337,11 @@ | |
| 337 | |
| 338 | # This works if and only if none of the symbol changesets are |
| 339 | # "backwards", hence our breaking of the backward changesets |
| 340 | # first, in the pre-hook. |
| 341 | |
| 342 | # Note that tag changesets cannot be backward as they don't |
| 343 | # have successors at all. |
| 344 | |
| 345 | # An interesting thing IMHO, is that after breaking the |
| 346 | # backward symbol changesets we should not have any circles |
| 347 | # any longer. Each circle which would still be present has to |
| 348 |