Developer forums (C::B DEVELOPMENT STRICTLY!) > CodeCompletion redesign

Patch: function arguments added to autocomplete tooltip

<< < (15/24) > >>

MortenMacFly:

--- Quote from: ollydbg on December 24, 2012, 08:19:55 am ---Can you make the comment panel's background color different from the code editor's background?

--- End quote ---
Why not just simply provide a border?

p2rkw:

--- Quote ---What does this do to performance? (Memory/CPU)
--- End quote ---
Plain documentation is stored in additional map, so memory isn't wasted when token haven't got documentation.
Html is generated on the fly from tokens generated (and cached) in CodeCompletion::CodeComplete, there aren't additional searches in token tree.  Most of the times token index is passed in link's href, so it can be obtained from tree without search.


--- Quote ---I second that.
--- End quote ---
And me too :) But first I have to find out how create patch that adds files to repo.


--- Quote ---Can you make the comment panel's background color different from the code editor's background?
currently, they are both white.
--- End quote ---
I forgot that default background color is white :) I already added border. I think that colors of background, text and links should be customizable.


--- Quote ---EDIT: it seems that click on the HTML link(in the comment panel) does not work.
--- End quote ---
Which exactly doesn't work? Can you try this code:

--- Code: ---struct ThisLinkWorks{};

//! A test class.
/*!
A more elaborate class description.
*/
class Test
{
public:
  //! An enum.
  /*! More detailed enum description. */
  enum TEnum {
    TVal1 = 2, /*!< Enum value TVal1. */
    TVal2, /*!< Enum value TVal2. */
    TVal3 /*!< Enum value TVal3. */
  }
  //! Enum pointer.
  /*! Details. */
  *enumPtr,
  //! Enum variable.
  /*! Details. */
  enumVar;
  //! A constructor.
  //! Constructing constructor of constructive Test class.
  /*!
  A more elaborate description of the constructor.
  */
  Test();
  //! A destructor.
  /*!
  A more elaborate description of the destructor.
  */
  ~Test();
  //! A normal member taking two arguments and returning an <font color=red>integer</font> value.
  /*!
  \param a an integer argument.
  \param s a constant character pointer.
  \return The test results
  \sa Test, ~Test, Test() testMeToo  publicVar
  */
  int testMe(int a, const char* s);
  //! A pure virtual member.
  /*!
  \sa testMe()
  \param c1 the first argument.
  \param c2 the second argument.
  */
  virtual void testMeToo(char c1, char c2) = 0;
  //! A public variable.
  /*!
  Details.
  */
  int publicVar;
  //! A function variable.
  /*!
  Details.
  */
  int (*handler)(int a, int b);

  ThisLinkWorks function4(ThisLinkDoesntWork tldw);
};
--- End code ---

PS. Can also try to implement methods using codecompletion?

MortenMacFly:

--- Quote from: p2rkw on December 26, 2012, 04:31:42 pm ---Which exactly doesn't work? Can you try this code:

--- End quote ---
Well in fact, on Windows none of the blue links works for me:
- Open Decl/Impl,
- Close
- Top
...what would be the expected result? I thought its maybe not yet implemented.

ollydbg:

--- Quote from: p2rkw on December 26, 2012, 04:31:42 pm ---
--- Quote ---I second that.
--- End quote ---
And me too :) But first I have to find out how create patch that adds files to repo.

--- End quote ---
If you use TortoiseSVN under Windows, just right click on the new file, and select "add" command. If you use SVN command, I believe there is a similar "add" command.
After that, the new file's content will be included in your patch.


--- Quote ---PS. Can also try to implement methods using codecompletion?

--- End quote ---
I don't understand this sentence. ???

oBFusCATed:

--- Quote from: ollydbg on December 24, 2012, 07:51:31 am ---2, [...] is it possible to add a new file (e.g. DoxygenParser.cpp)

--- End quote ---
Please stick to lower case file names.

Navigation

[0] Message Index

[#] Next page

[*] Previous page

Go to full version