Re: [wxperl-users] [bug] DocManager::MakeDefaultName() broken since v0.16

[Mattia B. <mat...@li...>]

Il Tue, 30 Sep 2003 18:35:37 +0100 Simon Flack <sf...@fl...> ha scritto:

  Hello,

> Here's the cvsweb diff of the change in question:
>   http://makeashorterlink.com/?U1CA41E06
> 
> The macro used for Wx::DocManager::MakeDefaultName in the current
> version 
> doesn't work.
  Sorry, fixed in CVS.

> MakeDefaultName() in perl differs from the C++
> implimentation 
> in that it returns the filename rather than modifying the arg
> passed in. I'm 
> happy for MakeDefaultName() to work the same as it does in
> wxWindows if you 
> can get it to work. i.e.:

>   my $i = 0;
>   sub MyDocManager::MakeDefaultName {
>       my $self = shift;
>       $_[0] = 'myname' . ++$i;
>   }
> 
> as opposed to pre-0.16:
>   my $i = 0;
>   sub MyDocManager::MakeDefaultName {
>       my $self = shift;
>       return 'myname' . ++$i;
>   }

  While I can get it to work, it'd be inconsistent with the rest of wxPerl.
What about this note in MakeDefaultName and GetPrintableName?

\perlnote{In wxPerl this function must return the modified name rather
than just modifying the argument.}

Suggestions for a better wording are really appreciated.

> I'm having another problem since version 0.16 in that controls
> inside 
> Wx::DocMDIChildFrames aren't repainted unless I resize the
> childframe. Might 
> be a problem in my code, especially since I'm having problems
> creating a 
> minimal demo of the problem. Will keep you posted on that.
  Might it be a change in wxWindows? Are you using the binaries I provide
or are you compiling wxPerl/wxWindows yourself?

Thanks!
Mattia