Duet3D Logo Duet3D
    • Tags
    • Documentation
    • Order
    • Register
    • Login

    Please improve the handling of TAB characters in the DWC editor

    Scheduled Pinned Locked Moved
    Duet Web Control wishlist
    6
    19
    683
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • zaptaundefined
      zapta @Baenwort
      last edited by

      @baenwort, of course, you can.

      And if you also want that pressing the delete key will insert a delete character and pressing the backspace key will insert a backspace character you can ask for it as well.

      😉

      1 Reply Last reply Reply Quote 2
      • zaptaundefined
        zapta @dc42
        last edited by

        @dc42 said in Please improve the handling of TAB characters in the DWC editor:

        @zapta IMO it makes no sense to have tabs positions set to every 8 columns in an editor.

        It seems that this is this forum system also assumes 8 chars per tab 😉

        Non indented
        	Indented by a single tab
        

        And the preview is rendered like this

        da00ae12-cfa3-4b9f-b9c1-8303b9b8024f-image.png

        The ambiguity of the tab character is especially problematic with languages such as RRF gcode whose semantic depends on indentation. That is for example why RRF complains about the code below because it mixes spaces and tabs in the same nested block

        if 1 < 2
          if 1 < 2   ; indented by two spaces
        	echo "aaa"  ; indented by a single tab
        

        6ae36eb8-d94e-4c2e-8fc8-7bc3ddbbc5e9-image.png

        Sticking to spaces eliminate this ambiguity.

        dc42undefined 1 Reply Last reply Reply Quote 0
        • dc42undefined
          dc42 administrators @zapta
          last edited by

          @zapta said in Please improve the handling of TAB characters in the DWC editor:

          Sticking to spaces eliminate this ambiguity.

          Sticking to tabs also eliminates it; but many people using programming languages that use indentation to indicate block scope prefer to use only spaces.

          Duet WiFi hardware designer and firmware engineer
          Please do not ask me for Duet support via PM or email, use the forum
          http://www.escher3d.com, https://miscsolutions.wordpress.com

          zaptaundefined 1 Reply Last reply Reply Quote 1
          • zaptaundefined
            zapta @dc42
            last edited by zapta

            @dc42 said in Please improve the handling of TAB characters in the DWC editor:

            Sticking to tabs also eliminates it;

            I don't see how it will eliminate the ambiguity, since the tab character doesn't have a standard interpretation and editors use 2, 4 or 8 positions.

            @dc42 said in Please improve the handling of TAB characters in the DWC editor:

            many people using programming languages that use indentation to indicate block scope prefer to use only spaces.

            I wonder if this is the case. For example, python is a popular indentation sensitive language and its official style guide says "Spaces are the preferred indentation method".

            https://www.python.org/dev/peps/pep-0008/#tabs-or-spaces,

            The Tab character used to have a single agreed upon interpretation (moving to next 8 positions column) but over the years people came with their own interpretations to the point that it's common these days to avoid it all together.

            BTW, this became an issue with DWC when conditional gcode was introduced. Before that we didn't use to indent.

            dc42undefined 1 Reply Last reply Reply Quote 0
            • dc42undefined
              dc42 administrators @zapta
              last edited by

              @zapta said in Please improve the handling of TAB characters in the DWC editor:

              I don't see how it will eliminate the ambiguity, since the tab character doesn't have a standard interpretation and editors use 2, 4 or 8 positions.

              Because for the purposes of indenting command blocks, all that matters is how many tabs you have at the start of the line; and it doesn't matter whether those tabs are displayed as 4, 8 or some other nonzero number of characters.

              Duet WiFi hardware designer and firmware engineer
              Please do not ask me for Duet support via PM or email, use the forum
              http://www.escher3d.com, https://miscsolutions.wordpress.com

              zaptaundefined 1 Reply Last reply Reply Quote 1
              • zaptaundefined
                zapta @dc42
                last edited by

                @dc42 said in Please improve the handling of TAB characters in the DWC editor:

                it doesn't matter whether those tabs are displayed as 4, 8 or some other nonzero number of characters.

                4 vs 8 will render the indentation differently to the user, right? Is this is why you warn if the same IF block contains both tabs and spaces? Because of the potential ambiguity?

                a3769ee9-73e1-492a-8e9c-e7d67a1eb201-image.png

                dc42undefined 1 Reply Last reply Reply Quote 0
                • dc42undefined
                  dc42 administrators @zapta
                  last edited by

                  @zapta said in Please improve the handling of TAB characters in the DWC editor:

                  4 vs 8 will render the indentation differently to the user, right? Is this is why you warn if the same IF block contains both tabs and spaces? Because of the potential ambiguity?

                  Yes and yes. It's only the indentation characters that need to be all spaces or all tabs so as to show the correct block scope in any editor. It doesn't matter whether the end-of-line comment is preceded by tabs, space, or a mixture - but of course if tabs are used, the editor tab setting will affect where the comment is displayed.

                  I usually use 2 spaces to indent a conditional GCode block, and tabs to place the end-of-line comments.

                  Duet WiFi hardware designer and firmware engineer
                  Please do not ask me for Duet support via PM or email, use the forum
                  http://www.escher3d.com, https://miscsolutions.wordpress.com

                  zaptaundefined 1 Reply Last reply Reply Quote 1
                  • zaptaundefined
                    zapta @dc42
                    last edited by

                    Today I learned that DWC inserts tabs even if I don't use the tab key.

                    To reproduce:

                    1. Start with a command indented by four spaces.
                    <four spaces>G1
                    
                    1. Position the cursor at the end of the line, press Enter, and type the second command, e.g. G2. Notice how the second line is indented with a single tab rather than four spaces.
                    <four spaces>G1
                    <tab>G2
                    

                    It seems that DWC not only supports tabs but actually prefers them over spaces.

                    gloomyandyundefined dc42undefined 2 Replies Last reply Reply Quote 0
                    • gloomyandyundefined
                      gloomyandy @zapta
                      last edited by

                      @zapta Yep I've also noticed it does this (and it is very irritating!).

                      Having to deal with files that contain mixed tabs/spaces for indentation is a personal bugbear of mine and creating them is in my view a terrible crime (though not as bad as the folks that reformat entire files to fit in with "the one true way" that they think the layout should look!). For many years now I've tried to practise (not always successfully!) an approach of using whatever style the original author has been using when making changes to existing source code. Personally I'd prefer to have some sort of option to use spaces for indentation (and to use the tab key to generate that indentation rather then insert a tab character), which is common in many other editors. Even better (IMHO) would be to have it auto adjust based on the exiting style(like some more advanced editors try to do, sometimes it works!).

                      1 Reply Last reply Reply Quote 0
                      • dc42undefined
                        dc42 administrators @zapta
                        last edited by

                        @zapta I've asked @chrishamm to look into this.

                        Duet WiFi hardware designer and firmware engineer
                        Please do not ask me for Duet support via PM or email, use the forum
                        http://www.escher3d.com, https://miscsolutions.wordpress.com

                        1 Reply Last reply Reply Quote 1
                        • zaptaundefined
                          zapta
                          last edited by

                          @dc42 said in Please improve the handling of TAB characters in the DWC editor:

                          @zapta I've asked @chrishamm to look into this.

                          Thanks.

                          1 Reply Last reply Reply Quote 0
                          • First post
                            Last post
                          Unless otherwise noted, all forum content is licensed under CC-BY-SA