Re: [Cpptool-develop] First Test of Addin ... and some bug report

[Baptiste L. <gai...@fr...>]

----- Original Message -----
From: "Baptiste Lepilleur" <gai...@fr...>
To: "CppTool Mailing List" <Cpp...@li...>
Sent: Wednesday, December 25, 2002 1:30 PM
Subject: Re: [Cpptool-develop] First Test of Addin ... and some bug report


> ----- Original Message -----
> From: "Andre Baresel" <and...@gm...>
> To: <gai...@fr...>; "CppTool Mailing List"
> <Cpp...@li...>
> Sent: Wednesday, December 25, 2002 1:04 PM
> Subject: Re: [Cpptool-develop] First Test of Addin ... and some bug report
>
>
> > Andre Baresel wrote:
> >
> > > Hello Baptiste,
> > >
> > > it was not a problem of the service pack. I'm using now also SP5.
> > > However the Bug on wrong replacement occures in the posted example if
> > > you run RenameTemp with the cursor on the declaration "int a = test;".
> > > Only in this case the replacement is done as reported in my first mail
> > > about this bug.
> > >
> > > Happy bug finding, I'm starting with InlineTemp !
> > >
> > > -- Andre
> > >
> > >
> > Sorry this bug description was wrong, I retested it !
> > The problem are the tabulator's and not the refactoring algorithm.
> > if tabulators are in a program the textdocument seems to replace the
> > text in wrong positions !
> > So we have to think about how Tabs can be converted to correct
> > positions. Maybe it is possible to
> > access the settings of the visual studio to find the length of a tab
> > that is currently used.
>
> That would explain why I failed to reproduce it. A simple work around for
> now is to use the option 'insert space instead of space'. Will need to
check
> if it is even possible to access this configuration setting.

>
> This will impact the class rfta/LineBasedTextDocument which handle
> conversion from 'index of the character from the beginning of the file' to
> 'line,column of the character'.


Well, I found how to get the tab size setting. It was a TextDocument
property. I added the code to the wrapper. LineBasedTextDocument will need
to be updated to handle this.

Baptiste.

> Baptiste.