Skip to content

Commit 0207147

Browse files
authored
Merge pull request #48 from meteokid/cstylefix
Fix multi line --enable-replacements --c-relations
2 parents ebba129 + 6fb9180 commit 0207147

File tree

2 files changed

+18
-16
lines changed

2 files changed

+18
-16
lines changed

fprettify/__init__.py

+12-12
Original file line numberDiff line numberDiff line change
@@ -628,19 +628,19 @@ def replace_relational_single_fline(f_line, cstyle):
628628
if not STR_OPEN_RE.match(part):
629629
# also exclude / if we see a namelist and data statement
630630
if cstyle:
631-
part = re.sub(r"\s*\.LT\.\s*", "<", part, flags=RE_FLAGS)
632-
part = re.sub(r"\s*\.LE\.\s*", "<=", part, flags=RE_FLAGS)
633-
part = re.sub(r"\s*\.GT\.\s*", ">", part, flags=RE_FLAGS)
634-
part = re.sub(r"\s*\.GE\.\s*", ">=", part, flags=RE_FLAGS)
635-
part = re.sub(r"\s*\.EQ\.\s*", "==", part, flags=RE_FLAGS)
636-
part = re.sub(r"\s*\.NE\.\s*", "/=", part, flags=RE_FLAGS)
631+
part = re.sub(r"\.LT\.", "< ", part, flags=RE_FLAGS)
632+
part = re.sub(r"\.LE\.", "<= ", part, flags=RE_FLAGS)
633+
part = re.sub(r"\.GT\.", "> ", part, flags=RE_FLAGS)
634+
part = re.sub(r"\.GE\.", ">= ", part, flags=RE_FLAGS)
635+
part = re.sub(r"\.EQ\.", "== ", part, flags=RE_FLAGS)
636+
part = re.sub(r"\.NE\.", "/= ", part, flags=RE_FLAGS)
637637
else:
638-
part = re.sub(r"\s*<=\s*", ".le.", part, flags=RE_FLAGS)
639-
part = re.sub(r"\s*<\s*", ".lt.", part, flags=RE_FLAGS)
640-
part = re.sub(r"\s*>=\s*", ".ge.", part, flags=RE_FLAGS)
641-
part = re.sub(r"\s*>\s*", ".gt.", part, flags=RE_FLAGS)
642-
part = re.sub(r"\s*==\s*", ".eq.", part, flags=RE_FLAGS)
643-
part = re.sub(r"\s*\/=\s*", ".ne.", part, flags=RE_FLAGS)
638+
part = re.sub(r"<=", ".le.", part, flags=RE_FLAGS)
639+
part = re.sub(r"<", ".lt.", part, flags=RE_FLAGS)
640+
part = re.sub(r">=", ".ge.", part, flags=RE_FLAGS)
641+
part = re.sub(r">", ".gt.", part, flags=RE_FLAGS)
642+
part = re.sub(r"==", ".eq.", part, flags=RE_FLAGS)
643+
part = re.sub(r"\/=", ".ne.", part, flags=RE_FLAGS)
644644

645645
line_parts[pos] = part
646646

fprettify/tests/__init__.py

+6-4
Original file line numberDiff line numberDiff line change
@@ -368,7 +368,8 @@ def test_relation_replacement(self):
368368
"if( min <= max .and. min .le. thres)",
369369
"'==== heading",
370370
"if (vtk%my_rank .eq. 0) write (vtk%filehandle_par, '(\"<DataArray",
371-
"'(\"</Collection>\","]
371+
"'(\"</Collection>\",",
372+
"if (abc(1) .lt. -bca .or. &\n qwe .gt. ewq) then"]
372373
f_outstring = ["if (min .lt. max .and. min .lt. thres)",
373374
"if (min .gt. max .and. min .gt. thres)",
374375
"if (min .eq. max .and. min .eq. thres)",
@@ -377,7 +378,8 @@ def test_relation_replacement(self):
377378
"if (min .le. max .and. min .le. thres)",
378379
"'==== heading",
379380
"if (vtk%my_rank .eq. 0) write (vtk%filehandle_par, '(\"<DataArray",
380-
"'(\"</Collection>\","]
381+
"'(\"</Collection>\",",
382+
"if (abc(1) .lt. -bca .or. &\n qwe .gt. ewq) then"]
381383
c_outstring = ["if (min < max .and. min < thres)",
382384
"if (min > max .and. min > thres)",
383385
"if (min == max .and. min == thres)",
@@ -386,8 +388,8 @@ def test_relation_replacement(self):
386388
"if (min <= max .and. min <= thres)",
387389
"'==== heading",
388390
"if (vtk%my_rank == 0) write (vtk%filehandle_par, '(\"<DataArray",
389-
"'(\"</Collection>\","]
390-
391+
"'(\"</Collection>\",",
392+
"if (abc(1) < -bca .or. &\n qwe > ewq) then"]
391393
for i in range(0, len(instring)):
392394
self.assert_fprettify_result(['--enable-replacements', '--c-relations'], instring[i], c_outstring[i])
393395
self.assert_fprettify_result(['--enable-replacements'], instring[i], f_outstring[i])

0 commit comments

Comments
 (0)