Community
    • Login

    Plugin built with VS 2013 Express vs. Plugin built with VS 2013 Community

    Scheduled Pinned Locked Moved Notepad++ & Plugin Development
    24 Posts 2 Posters 22.6k Views
    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.
    • YaronY
      Yaron
      last edited by

      Hello Claudia,

      Thank you so much. Highly appreciated.

      My long shot was a difference between building the DLL on 32 or 64.

      Debug/Release: I meant to ask if the default setting on opening the file was Debug.

      I don’t get any errors or warnings.

      I download the current zip (master branch).
      If I hover over an older commit’s <> button (Browse the repository at this point in the history), download the up-to-then source code and build it - I get correct results.

      Best regards.

      Claudia FrankC 1 Reply Last reply Reply Quote 0
      • Claudia FrankC
        Claudia Frank @Yaron
        last edited by

        Hello Yaron

        My long shot was a difference between building the DLL on 32 or 64.

        I don’t think so, but bugs could be anywhere.

        Debug/Release: I meant to ask if the default setting on opening the file was Debug.

        Yes, as this it the main profile a developer works with.

        I don’t get any errors or warnings.
        I download the current zip (master branch).
        If I hover over an older commit’s <> button (Browse the repository at this point in the history), download the up-to-then source code and >build it - I get correct results.

        Which commit did you use? What about if you use the latest one?

        Cheers
        Claudia

        1 Reply Last reply Reply Quote 0
        • YaronY
          Yaron
          last edited by

          Hello Claudia,

          Thanks again. :)

          Using the latest commit - wrong results.

          A few minuets ago I downloaded the source up to this commit and got correct results.

          I think the problem started here. I can test again and verify it.

          Best regards.

          1 Reply Last reply Reply Quote 0
          • Claudia FrankC
            Claudia Frank
            last edited by

            Yaron, could you post your compare.ini?

            Cheers
            Claudia

            1 Reply Last reply Reply Quote 0
            • YaronY
              Yaron
              last edited by

              Hello Claudia,

              Compare.ini.

              Thank you very much.

              BR

              1 Reply Last reply Reply Quote 0
              • YaronY
                Yaron
                last edited by

                Hello again,

                The problem starts in this commit.

                Thank you.

                BR

                1 Reply Last reply Reply Quote 0
                • Claudia FrankC
                  Claudia Frank
                  last edited by

                  This is were I currently investigate, to be more precise I assume the issue in addBlankSection.

                  Cheers
                  Claudia

                  1 Reply Last reply Reply Quote 0
                  • YaronY
                    Yaron
                    last edited by

                    You’re amazing.

                    On one hand I think that it’s too much, but on the other hand I don’t want you to stop. :)

                    I was about to start investigating it myself; I was overwhelmed by the scale of the unfamiliar code.

                    Thank you very much!

                    BR

                    1 Reply Last reply Reply Quote 0
                    • Claudia FrankC
                      Claudia Frank
                      last edited by Claudia Frank

                      It might be that this was a hasty reaction as it could be that the function already got the wrong value.
                      What I can say so far is that the inserttext from addBlankSection

                      ::SendMessage(window, SCI_INSERTTEXT, posAdd, (LPARAM)buff.data());
                      

                      only gets one '\r\n”

                      {'text': '\r\n',            'modificationType': 0x11,      'linesAdded': 1, 'length': 2, 'position': 3, 'line': 0 }
                      

                      whereas my dll gets three

                      {'text': '\r\n\r\n\r\n',    'modificationType': 0x11,      'linesAdded': 3, 'length': 6, 'position': 3, 'line': 0 }
                      

                      Test was done with two files, one having

                      1
                      5
                      

                      whereas the other has

                      1
                      2
                      3
                      4
                      5
                      

                      It is most likely that I need your help to find out what the problem is, meaning
                      you need to do some debugging as well. But not for today, I need to have a rest.
                      Maybe we can go on tomorrow?

                      Cheers
                      Claudia

                      1 Reply Last reply Reply Quote 0
                      • YaronY
                        Yaron
                        last edited by

                        Good morning Claudia,

                        I hope you’re reading these lines after having a good and refreshing rest.

                        I find it difficult to enlarge the variety of my thanks expressions.
                        So, many thanks again. :)

                        I’ve never used the Debugger.
                        Bear with me.

                        Have a great day.

                        1 Reply Last reply Reply Quote 0
                        • Claudia FrankC
                          Claudia Frank
                          last edited by Claudia Frank

                          Hello Yaron,

                          The problem is generated by a compiler bug. It tries to optimize a little bit too much. ;-)

                          Either disable optimization, by right clicking on compare in project explorer,
                          properties -> C/C++ -> all options -> Optimization and change its value

                          or the preferred way

                          update your visual studio 2013 to latest version. ;-)

                          Cheers
                          Claudia

                          1 Reply Last reply Reply Quote 0
                          • YaronY
                            Yaron
                            last edited by

                            Hello Claudia,

                            You never cease to amaze me.
                            I take my hat off. Oracle Claudia!

                            Thank you so much. I’m truly grateful.

                            Could you please enlighten me and explain briefly how you got it?
                            I hope it’s not a trade secret. :)

                            Best regards.

                            1 Reply Last reply Reply Quote 0
                            • Claudia FrankC
                              Claudia Frank
                              last edited by

                              Hello Yaron,

                              no, it isn’t a trade secret ;-) - but it wasn’t really a clever job, it was just an eliminating process and LUCK.

                              I only tell you the steps which lead to the finding not the many wasted miserable steps I’ve done before. ;-)

                              After I didn’t find any obvious issues which could explain reported behavior I thought about the differences of your
                              and mine environment. After that I setup an windows 7 32 bit with visual studio 2013. Downloaded source and compiled it.
                              Tested it and it worked. Even more confused as I expected it to fail. Thought about it again and started comparing yours and mine.
                              Same W7, same VS2013, same source -> different results.
                              Same W7 (?) -> didn’t install all updates -> didn’t use Hebrew = keep in mind
                              VS2013 -> same
                              source-> same, you did even test it with different npp version as I read.
                              So it must be W7 -> why??? Did a search on the internet about reporting problem with VS2013 and non-english language.
                              Results, but nothing pointing to such an issue.
                              Then I did a search about problems with visual studio and having different language. And while scanning the result
                              I saw one who had an issue installing an update - that’s where I started thinking about different version.
                              I had VS2013 update 4 already so I gave it a chance and -> compiled the source and tested it -> problem was the same.

                              You see, no rocket science.

                              Cheers
                              Claudia

                              1 Reply Last reply Reply Quote 0
                              • YaronY
                                Yaron
                                last edited by

                                Hello Claudia,

                                Thank you for the explanation. You know, men can get curious too. :)

                                And now I just can’t help it.
                                After explaining the process of reasoning leading to her famous deductions, Shirley Holmes’ insights seemed to lose some of their former glamour.
                                But then on a second thought the shine became even brighter.

                                Well, I assumed you had performed some searches and tests.
                                Now that I know the scale of those actions, my gratitude grows and deepens.

                                And finally, rocket science would be no “rocket science” for you. :)

                                Best regards.

                                1 Reply Last reply Reply Quote 0
                                • First post
                                  Last post
                                The Community of users of the Notepad++ text editor.
                                Powered by NodeBB | Contributors