Remove preproc fromdetect_fixed_format and handle line continuation

Additionally, `test_helper.py` has been removed since all the test cases in there are in the examples of the docstring and hence they are covered by doctest already.

Author: albertziegenhagel

fortls/helper_functions.py

@@ -38,7 +38,7 @@ def expand_name(line: str, char_pos: int) -> str:
     return ""
 
 
-def detect_fixed_format(file_lines: list[str], preproc: bool = False) -> bool:
+def detect_fixed_format(file_lines: list[str]) -> bool:
     """Detect fixed/free format by looking for characters in label columns
     and variable declarations before column 6. Treat intersection format
     files as free format.
@@ -47,9 +47,6 @@ def detect_fixed_format(file_lines: list[str], preproc: bool = False) -> bool:
     ----------
     file_lines : list[str]
         List of consecutive file lines
-    preproc : bool
-        If true, preprocessor directives (lines starting with '#') will be
-        ignored
 
     Returns
     -------
@@ -72,13 +69,29 @@ def detect_fixed_format(file_lines: list[str], preproc: bool = False) -> bool:
     >>> detect_fixed_format(['trailing line & ! comment'])
     False
 
-    But preprocessor lines might be ignored
-    >>> detect_fixed_format(['#if defined(A) && !defined(B)'], preproc=True)
+    But preprocessor lines will be ignored
+    >>> detect_fixed_format(
+    ...     ['#if defined(A) && !defined(B)', 'C Fixed format', '#endif'])
     True
+
+    >>> detect_fixed_format(
+    ...     ['#if defined(A) && !defined(B)', ' free format', '#endif'])
+    False
+
+    And preprocessor line-continuation is taken into account
+    >>> detect_fixed_format(
+    ...     ['#if defined(A) \\\\ ', ' && !defined(B)', 'C Fixed format', '#endif'])
+    True
+
+    >>> detect_fixed_format(
+    ...     ['#if defined(A) \\\\', '&& \\\\', '!defined(B)', ' free format', '#endif'])
+    False
     """
+    pp_continue = False
     for line in file_lines:
         # Ignore preprocessor lines
-        if preproc and line.startswith("#"):
+        if line.startswith("#") or pp_continue:
+            pp_continue = line.rstrip().endswith("\\")
             continue
         if FRegex.FREE_FORMAT_TEST.match(line):
             return False

fortls/parse_fortran.py

@@ -904,7 +904,7 @@ def load_from_disk(self) -> tuple[str | None, bool | None]:
 
             self.hash = hash
             self.contents_split = contents.splitlines()
-            self.fixed = detect_fixed_format(self.contents_split, self.preproc)
+            self.fixed = detect_fixed_format(self.contents_split)
             self.contents_pp = self.contents_split
             self.nLines = len(self.contents_split)
             return None, True
@@ -1013,7 +1013,7 @@ def set_contents(self, contents_split: list, detect_format: bool = True):
         self.contents_pp = self.contents_split
         self.nLines = len(self.contents_split)
         if detect_format:
-            self.fixed = detect_fixed_format(self.contents_split, self.preproc)
+            self.fixed = detect_fixed_format(self.contents_split)
 
     def get_line(self, line_no: int, pp_content: bool = False) -> str:
         """Get single line from file"""