Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SCRIPT_FORMAT_UNCHANGED bugs/glitches #7

Open
2 tasks done
wipe2238 opened this issue Mar 4, 2020 · 0 comments
Open
2 tasks done

SCRIPT_FORMAT_UNCHANGED bugs/glitches #7

wipe2238 opened this issue Mar 4, 2020 · 0 comments
Labels
bug Something isn't working
Milestone

Comments

@wipe2238
Copy link
Member

wipe2238 commented Mar 4, 2020

  • When 2nd+ argument becomes 1st, it keeps leading whitespace which is it's most likely unwanted. Leading spacing used in original 1st argument should be probably cached and applied in following scenario.
@@ -272,8 +272,8 @@
 end
 procedure Node2217a begin
    if ((item_caps_total(dude_obj)) >= 100) then begin
-      item_caps_adjust(dude_obj, -100);
-      item_caps_adjust(self_obj, 100);
+      dude_caps_adjust( -100);
+      self_caps_adjust( 100);
       critter_heal(dude_obj, -((get_critter_stat(dude_obj,(35))) - (get_critter_stat(dude_obj,(7)))));
       call Node022;
    end else begin
  • Moving argument to end gives ugly results as well
3:1 = RunAfter IfFileName:0301.ssl IfFunction:f DoArgumentsMoveBack:0
@@ : fileline<0301.ssl:1>
<- f(a, b, c);
-> f(b, c,a);

#6

@wipe2238 wipe2238 added the bug Something isn't working label Mar 4, 2020
@wipe2238 wipe2238 added this to the Release milestone Mar 4, 2020
wipe2238 added a commit that referenced this issue Apr 13, 2020
  tests are always included in build, but runs on demand only
  cmake --build <build_dir> --target ReDefine.Test
- added mingw support, for internal use
  uses c++17 due to missing std::experimental::filesystem
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant