Added comment on why the ID was changed from u_uint to c_ulong

hardbyte · Jul 24, 2018 · 958f57c · 958f57c
1 parent 5a4868e
commit 958f57c
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/can/interfaces/pcan/basic.py b/can/interfaces/pcan/basic.py
@@ -292,7 +292,7 @@ class TPCANMsg (Structure):
     """
     Represents a PCAN message
     """
-    _fields_ = [ ("ID",      c_ulong),          # 11/29-bit message identifier
+    _fields_ = [ ("ID",      c_ulong),          # 11/29-bit message identifier - ID was changed from u_uint to c_ulong, as it caused a "Segmentation fault: 11" on Mac
                  ("MSGTYPE", TPCANMessageType), # Type of the message
                  ("LEN",     c_ubyte),          # Data Length Code of the message (0..8)
                  ("DATA",    c_ubyte * 8) ]     # Data of the message (DATA[0]..DATA[7])
@@ -312,7 +312,7 @@ class TPCANMsgFD (Structure):
     """
     Represents a PCAN message
     """
-    _fields_ = [ ("ID",      c_ulong),          # 11/29-bit message identifier
+    _fields_ = [ ("ID",      c_ulong),          # 11/29-bit message identifier - ID was changed from u_uint to c_ulong, as it caused a "Segmentation fault: 11" on Mac
                  ("MSGTYPE", TPCANMessageType), # Type of the message
                  ("DLC",     c_ubyte),          # Data Length Code of the message (0..15)
                  ("DATA",    c_ubyte * 64) ]    # Data of the message (DATA[0]..DATA[63])