[haiku-commits] Re: r36656 - haiku/trunk/src/apps/stylededit

  • From: Ingo Weinhold <ingo_weinhold@xxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Thu, 06 May 2010 23:10:19 +0200

On 2010-05-06 at 17:35:51 [+0200], axeld@xxxxxxxxxxxxxxxx wrote:
> Author: axeld
> Date: 2010-05-06 17:35:50 +0200 (Thu, 06 May 2010)
> New Revision: 36656
> Changeset: http://dev.haiku-os.org/changeset/36656/haiku
> 
> Modified:
>    haiku/trunk/src/apps/stylededit/StyledEditWindow.cpp
>    haiku/trunk/src/apps/stylededit/StyledEditWindow.h
> Log:
> * Applied patch by x-ist that adds StyledEdit storing its window position 
> and
>   size per document in a se-info attribute.
> * Minor style changes.
> 
> 
> Modified: haiku/trunk/src/apps/stylededit/StyledEditWindow.cpp
> ===================================================================
> --- haiku/trunk/src/apps/stylededit/StyledEditWindow.cpp    2010-05-06 
> 15:27:25 UTC (rev 36655)
> +++ haiku/trunk/src/apps/stylededit/StyledEditWindow.cpp    2010-05-06 
> 15:35:50 UTC (rev 36656)
> @@ -44,9 +46,12 @@
>  
>  const float kLineViewWidth = 30.0;
>  
> +#define ATTRNAME_SE_INFO "se-info"

Do attribute names have to be cryptic?

>  void
> +StyledEditWindow::LoadAttrs()
> +{
> +    if (!fSaveMessage)
> +        return;

Do I want to know why the window has this mystic fSaveMessage attribute or 
better not?

CU, Ingo

Other related posts: