History log of /freebsd/usr.sbin/bsdinstall/scripts/wlanconfig (Results 26 – 50 of 123)
Revision (<<< Hide revision tags) (Show revision tags >>>) Date Author Comments
# 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 ...


12345