Hello Albert,<br><br>just two quick comments.<br><br>We are trying to 
get away from using sal_Int16 for sheet numbers and use SCTAB instead, 
so it would be nice if you could change this.<br>And I think we can&#39;t/shouldn&#39;t create a document without a sheet (but Kohei may prove me wrong). And did you check that no negative numbers/characters are inserted?<br>

<br>
But all in all some good work. I think Kohei will take a closer look and give you some better advice.<br>
<br>
Regards,<br>
Markus<br><br>P.S. I prefer static_cast over c style casts, but don&#39;t know Kohei&#39;s opinion on this<br><br><div class="gmail_quote">2011/5/31 Albert Thuswaldner <span dir="ltr">&lt;<a href="mailto:albert.thuswaldner@gmail.com">albert.thuswaldner@gmail.com</a>&gt;</span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hi all,<br>
Submitting a patch for  Bug 33293 for review. Please let me know if<br>
you want me to improve some parts of it.<br>
<br>
Once you are happy with it you can commit the patch under the terms of<br>
MPL 1.1 / GPLv3+ / LGPLv3+<br>
triple license.<br>
<br>
To implement this feature request I have:<br>
1.) created a new config tab page called &quot;Initialize&quot;,were the user<br>
can set the desired number of sheets to start with.<br>
2.) created a new handler class  :ScTpInitOptions (tpinit.cxx/hxx) to<br>
deal with this config item.<br>
3.) modded the class (docoptio.cxx/hxx) to include this option<br>
4.) modded ScTabViewShell class (tabvwsh4.cxx) where the number of<br>
sheets actually are set<br>
<br>
Regards,<br>
<font color="#888888"><br>
/Albert<br>
</font><div><div></div><div class="h5"><br>
<br>
<br>
On Wed, May 18, 2011 at 08:39, Albert Thuswaldner<br>
&lt;<a href="mailto:albert.thuswaldner@gmail.com">albert.thuswaldner@gmail.com</a>&gt; wrote:<br>
&gt; Hi Kohei,<br>
&gt; Thanks for your help.<br>
&gt;<br>
&gt; On Mon, May 9, 2011 at 21:19, Kohei Yoshida &lt;<a href="mailto:kyoshida@novell.com">kyoshida@novell.com</a>&gt; wrote:<br>
&gt;&gt; On Sun, 2011-05-08 at 17:09 +0200, Albert Thuswaldner wrote:<br>
&gt;&gt;&gt; Dear List,<br>
&gt;&gt;&gt; As a first attempt to contribute some code to LO, I&#39;ve started hacking<br>
&gt;&gt;&gt; on his:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; <a href="https://bugs.freedesktop.org/show_bug.cgi?id=33293" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=33293</a><br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; I think I have come quite far with implementing the changes to the<br>
&gt;&gt;&gt; configuration dialog (see the patch). Here there is still some things<br>
&gt;&gt;&gt; to do, like figuring out why the contents of the tab page isn&#39;t<br>
&gt;&gt;&gt; showing up. Any hints?<br>
&gt;&gt;<br>
&gt;&gt; So, when you say the contents of the tag page isn&#39;t showing up, do you<br>
&gt;&gt; mean the page entry is not in the tree on the left at all, or the entry<br>
&gt;&gt; is there but the page shows up empty?<br>
&gt;<br>
&gt; The page entry was in the tree but the tab page itself showed up<br>
&gt; empty. However, I have solved this part now! :)<br>
&gt; There was an additional place in the code that needed to be modified.<br>
&gt;<br>
&gt;&gt;&gt; Also why are many of the same resource id&#39;s defined both in<br>
&gt;&gt;&gt; libs-core/svx/source/src/app.hrc and<br>
&gt;&gt;&gt; libs-core/svx/inc/svx/dialogs.hrc?<br>
&gt;&gt;<br>
&gt;&gt; I have no clue about this.  Maybe it&#39;s a remnant of one of those code<br>
&gt;&gt; relocation efforts that took place awhile ago in the OOo code?<br>
&gt;&gt;<br>
&gt;&gt;&gt; For the other part of problem i.e, how to actually set the number of<br>
&gt;&gt;&gt; sheets for a new spreadsheet, I am completely lost. I haven&#39;t found<br>
&gt;&gt;&gt; where in the code this should be done, (using opengrok.+ guessing).<br>
&gt;&gt;&gt; Could someone please give me a hint on how to go about finding this?<br>
&gt;&gt;&gt; How is a new spreadsheet generated? xml template? through using the<br>
&gt;&gt;&gt; internal api?<br>
&gt;&gt;<br>
&gt;&gt; Check around the ScTable class.  That&#39;s the class that implements each<br>
&gt;&gt; sheet instance.  Especially pay close attention to its constructor, and<br>
&gt;&gt; assuming that it gets called three times during the startup of Calc, you<br>
&gt;&gt; can perhaps set a break point in there to see who instaniates it 3<br>
&gt;&gt; times, and trace back from there.<br>
&gt;<br>
&gt; I&#39;ve mostly focused on the configuration part as of yet, now I will<br>
&gt; focus on this part.<br>
&gt; Thanks for the hint.<br>
&gt;<br>
&gt;&gt; Great to see you working on this, BTW.<br>
&gt;&gt;<br>
&gt;&gt; Kohei<br>
&gt;&gt;<br>
&gt;&gt; --<br>
&gt;&gt; Kohei Yoshida, LibreOffice hacker, Calc<br>
&gt;&gt; &lt;<a href="mailto:kyoshida@novell.com">kyoshida@novell.com</a>&gt;<br>
&gt;&gt;<br>
&gt;<br>
&gt; /Albert<br>
&gt;<br>
</div></div><br>_______________________________________________<br>
LibreOffice mailing list<br>
<a href="mailto:LibreOffice@lists.freedesktop.org">LibreOffice@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/libreoffice" target="_blank">http://lists.freedesktop.org/mailman/listinfo/libreoffice</a><br>
<br></blockquote></div><br>