update_ini: improve regex for finding the DISPLAY section

Fixes the bugs
 - if there is a commented DISPLAY sections before the DISPLAY section, the comments are used instead
 - DISPLAY section must not be at the end --> empty section

Author: hansu

src/emc/ini/update_ini.py

@@ -446,7 +446,7 @@ def insert_after(key, new_line, section):
     inistring, newini, all_sections = ini_preamble()
 
     all_sections.remove("DISPLAY")
-    section = re.search(r"\[DISPLAY\](.+?\n)(?=\[)", inistring, re.DOTALL)
+    section = re.search(r"(?:^|\n)[ \t]*(?!#)\[DISPLAY\](.+?\n)(?=\[|$)", inistring, re.DOTALL)
 
     if section: section = section.group(1)
     newini.write("\n[DISPLAY]")
@@ -495,10 +495,6 @@ def insert_after(key, new_line, section):
     # ~rename [TRAJ, DISPLAY]MIN_VELOCITY --> MIN_LINEAR_VELOCITY~
     # copy [TRAJ]MAX_LINEAR_VELOCITY -> [DISPLAY]MAX_LINEAR_VELOCITY
 
-    # Known bugs:
-    # - if there is a commented DISPLAY sections before the DISPLAY section, the comments are used
-    # - DISPLAY section must not be at the end --> empty section
-    
     #These sections don't need any work.
     copysection("FILTER")
     copysection("RS274NGC")