Merge pull request #1603 from peternewman/master-resync

Master resync

Author: peternewman

.codespellignore .codespellignorelines.codespellignore renamed to .codespellignorelines

@@ -0,0 +1,3 @@
+thead
+acn
+ACN

.codespellignorewords

@@ -12,7 +12,8 @@ COVERITY_SCAN_BUILD_URL="https://scan.coverity.com/scripts/travisci_build_coveri
 PYCHECKER_BLACKLIST="threading,unittest,cmd,optparse,google,google.protobuf,ssl,fftpack,lapack_lite,mtrand"
 
 SPELLINGBLACKLIST=$(cat <<-BLACKLIST
-      -wholename "./.codespellignore" -or \
+      -wholename "./.codespellignorewords" -or \
+      -wholename "./.codespellignorelines" -or \
       -wholename "./.git/*" -or \
       -wholename "./aclocal.m4" -or \
       -wholename "./config/config.guess" -or \
@@ -171,10 +172,10 @@ elif [[ $TASK = 'codespell' ]]; then
       $SPELLINGBLACKLIST \
       \) | xargs")
   # count the number of codespell errors
-  spellingerrors=$(zrun codespell --check-filenames --check-hidden --quiet 2 --regex "[a-zA-Z0-9][\\-'a-zA-Z0-9]+[a-zA-Z0-9]" --exclude-file .codespellignore $spellingfiles 2>&1 | wc -l)
+  spellingerrors=$(zrun codespell --check-filenames --check-hidden --quiet 2 --regex "[a-zA-Z0-9][\\-'a-zA-Z0-9]+[a-zA-Z0-9]" --exclude-file .codespellignorelines --ignore-words .codespellignorewords $spellingfiles 2>&1 | wc -l)
   if [[ $spellingerrors -ne 0 ]]; then
     # print the output for info
-    zrun codespell --check-filenames --check-hidden --quiet 2 --regex "[a-zA-Z0-9][\\-'a-zA-Z0-9]+[a-zA-Z0-9]" --exclude-file .codespellignore $spellingfiles
+    zrun codespell --check-filenames --check-hidden --quiet 2 --regex "[a-zA-Z0-9][\\-'a-zA-Z0-9]+[a-zA-Z0-9]" --exclude-file .codespellignorelines --ignore-words .codespellignorewords $spellingfiles
     echo "Found $spellingerrors spelling errors via codespell"
     exit 1;
   else

.travis-ci.sh

@@ -196,7 +196,11 @@ string NackReasonToString(uint16_t reason) {
     case NR_ACTION_NOT_SUPPORTED:
       return "Action not supported";
     case NR_ENDPOINT_NUMBER_INVALID:
-      return "Invalid endpoint";
+      return "Endpoint number invalid";
+    case NR_INVALID_ENDPOINT_MODE:
+      return "Invalid endpoint mode";
+    case NR_UNKNOWN_UID:
+      return "Unknown UID";
     default:
       ostringstream str;
       str << "Unknown, was " << reason;
@@ -786,7 +790,7 @@ string SlotInfoToString(uint8_t slot_type, uint16_t slot_label) {
       case SD_COLOR_SCROLL:
         return "Primary, scroll";
       case SD_COLOR_SEMAPHORE:
-        return "Primary, color semaphone";
+        return "Primary, color semaphore";
       case SD_COLOR_ADD_AMBER:
         return "Primary, additive amber";
       case SD_COLOR_ADD_WHITE:

common/rdm/RDMHelper.cpp

@@ -435,6 +435,7 @@ void CapitalizeLabel(string *s) {
 }
 
 void CustomCapitalizeLabel(string *s) {
+  // Remember to update the Doxygen in include/ola/StringUtils.h too
   static const char* const transforms[] = {
     "dhcp",
     "dmx",
@@ -443,6 +444,7 @@ void CustomCapitalizeLabel(string *s) {
     "ipv4",  // Should really be IPv4 probably, but better than nothing
     "ipv6",  // Should really be IPv6 probably, but better than nothing
     "led",
+    "mdmx",  // City Theatrical, should really be mDMX, but better than nothing
     "pdl",
     "pid",
     "rdm",

common/utils/StringUtils.cpp

@@ -932,7 +932,7 @@ fi
 AC_SUBST(PLUGIN_LIBS)
 AC_SUBST(OLA_SERVER_LIBS)
 
-# html & data directories
+# HTML & data directories
 ola_datadir=$datadir/olad
 www_datadir=$ola_datadir/www
 piddatadir=$datadir/ola/pids
@@ -1014,5 +1014,5 @@ Now type 'make @<:@<target>@:>@'
   where the optional <target> is:
     all          - build everything
     check        - run the tests
-    doxygen-doc  - generate the html documentation
+    doxygen-doc  - generate the HTML documentation
 -------------------------------------------------------"

configure.ac

@@ -441,8 +441,16 @@ void CapitalizeLabel(std::string *s);
  *
  * @param s a string to transform.
  * The following are capitalized:
- *   - ip
+ *   - dhcp
  *   - dmx
+ *   - dns
+ *   - ip
+ *   - ipv4
+ *   - ipv6
+ *   - led
+ *   - mdmx
+ *   - rdm
+ *   - uid
  */
 void CustomCapitalizeLabel(std::string *s);
 

include/ola/StringUtils.h

@@ -326,6 +326,11 @@ typedef enum {
 } rdm_data_type;
 
 
+// Remember to update the other places when adding new NACK codes:
+// common/rdm/RDMHelper.cpp
+// plugins/usbpro/DmxTriWidget.cpp
+// plugins/usbpro/DmxTriWidget.h
+// python/ola/OlaClient.py
 typedef enum {
   NR_UNKNOWN_PID = 0x0000,
   NR_FORMAT_ERROR = 0x0001,
@@ -339,7 +344,9 @@ typedef enum {
   NR_SUB_DEVICE_OUT_OF_RANGE = 0x0009,
   NR_PROXY_BUFFER_FULL = 0x000A,
   NR_ACTION_NOT_SUPPORTED = 0x000B,
-  NR_ENDPOINT_NUMBER_INVALID = 0x0011
+  NR_ENDPOINT_NUMBER_INVALID = 0x000C,
+  NR_INVALID_ENDPOINT_MODE = 0x000D,
+  NR_UNKNOWN_UID = 0x000E
 } rdm_nack_reason;
 
 

include/ola/rdm/RDMEnums.h

@@ -34,7 +34,7 @@ namespace web {
  * This is the base item class. Items are composed into sections.
  * Each item has the following:
  *  - A text description
- *  - A type, which controls how the item is renders
+ *  - A type, which controls how the item is rendered
  *  - The value.
  *  - Optional id. A non-empty id makes this item editable
  *  - Optional button text. Non-empty means this item gets it own button.

include/ola/web/JsonSections.h

@@ -320,7 +320,7 @@ class LibUsbAdaptor {
    *   output (depending on direction bits within bmRequestType)
    * @param wLength the length field for the setup packet. The data buffer
    *   should be at least this size.
-   * @param timeout timeout (in millseconds) that this function should wait
+   * @param timeout timeout (in milliseconds) that this function should wait
    *   before giving up due to no response being received. For an unlimited
    *   timeout, use value 0.
    * @returns on success, the number of bytes actually transferred