Dekel Tsur <[EMAIL PROTECTED]> writes:

| --J2SCkAp4GZ/dPZZf
| Content-Type: text/plain; charset=us-ascii
| 
| Selecting a bullet shape in the popup doesn't update the text field.
| A fix is attached.

As far as I can see, wrong fix.
(otoh I don't knwo the real problem)


| -     string getText() const;
| -     ///

instead of removing change to:
        string const & getText() const;
        
|       string getText();
|       ///
| -     char const * c_str() const;
| +     char const * c_str();

This should be const.

|  inline
| -string Bullet::getText() const
| -{
| -     return text;
| -}

inline
string const & Bullet::getText() const
{
        return text;
}

|  inline
| -char const * Bullet::c_str() const
| +char const * Bullet::c_str()
|  {
| -     return this->getText().c_str();
| +     return getText().c_str();
|  }

There should ideally be no need to change this.

| -     block<Bullet, 4> user_defined_bullets;
| +     mutable block<Bullet, 4> user_defined_bullets;

hmm, perhaps not needed?

        Lgb

Reply via email to