#
d5fddc4d |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Relying on dialog auto-sizing (width/height/rows = 0) is a mistake Use the provided API for calculating the appropriate size of menus
|
#
7599c854 |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Remove unnecessary quotes
|
#
1095816e |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Add missing quotes
|
#
01a2404c |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
In awk, if you're going to append a newline to your printf AND you're going to print only the argument, just use print
|
#
dba958af |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
This statement has too many backslashes
|
#
be34885e |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Neither printf (and as is commonly known) nor print need parens in awk
|
#
5943bcd1 |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Whitespace and alignment
|
#
191441ee |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
You don't need parentheses for awk's printf
|
#
0724c87c |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Continued resolution of conveluted statement We shouldn't be coding things like "x || (x && x) || x || x || x ..."
|
#
32c34b03 |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
These two error messages have always been backwards since inception
|
#
7f513d36 |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Why use $? when you can use the command itself
|
#
6b77f63e |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
If the first ping succeeded, why on Earth should we ping it again?
|
#
a1a6fec1 |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Start deconstructing a conveluted hunk of code
|
#
19cae84e |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Remove completely unnecesary parentheses
|
#
75199394 |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Why repeat yourself when you can send stderr to the same place as stdout?
|
#
66ef7d94 |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Properly quote variable
|
#
a5119576 |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Use more generic f_yesno() from provided API
|
#
f52733d4 |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
The output of dialog needs to be sanitized for portability/compatibility requirements
|
#
f20b7768 |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Whitespace alignment
|
#
d401d36c |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Sort the domains
|
#
95ee591e |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
The --no-items and --stdout options are non-standard and should be avoided
From the dialog(1) manual: using [--stdout] in portable scripts is not recommended
|
#
6e038cc2 |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
More efficiently make use of the exit status
|
#
1c612112 |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Stop repeating strings (centralize prompt string) NB: Changes to strings now only affect a single line
|
#
e2577019 |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Add missing backslash (no real effect; it's pedantic and correct for the interpolation level)
|
#
e7f2eb12 |
| 13-Dec-2016 |
Devin Teske <dteske@FreeBSD.org> |
Use the oft-neglected awk syntax "startcondition, stopcondition { ... }" to process the range of country labels which appear as columnar list from the "ifconfig DEV list countries" command. Not only
Use the oft-neglected awk syntax "startcondition, stopcondition { ... }" to process the range of country labels which appear as columnar list from the "ifconfig DEV list countries" command. Not only improving maintainability, but also properly encapsulating arguments in single-quotes instead of trying to escape whitespace. It is also completely unnecessary to collapse newlines into whitespace (shell will do this for you automatically upon expansion of the contents where necessary).
NB: This also changes the sorting algorithm to sort on the country code, not the country name. The type-ahead feature of dialog is destroyed if the tags are not sorted properly.
show more ...
|