Hi Pradeeban,
I have fixed the following issues:
1. Remove unused code and comments
2. Using UT_std_string_sprintf instead of sprintf
The following Table shows my Current State of My GSOC, please advise, thanks
Cross-platform
Windows GTK Cocoa
Make Frame expandable Finished Finished
Finished Starting
Frame SetHeight&SetWidth Finished Finished
60% Finished Starting
Table SetHeight&SetWidth Finished Finished
50% Finished Starting
Code Refactor 80%Finished 80%
Finished Starting
thanks,
Vincent
2012/7/10 Kathiravelu Pradeeban <kk.pradeeban@gmail.com>
>
> Hi Vincent,
> Mid evaluations are up online. You can start submitting. Can you send
> a complete summary of what you have done so far to the abiword dev
> mailing list? That will help us for the record, and in making the
> decision for the Summer of Code mid evaluations.
>
> Thank you.
> Regards,
> Pradeeban.
>
> On Tue, Jul 10, 2012 at 6:58 AM, Vincent Kang <vincent.abiword@gmail.com> wrote:
> > Hi Pradeeban,
> >
> > Thanks for review part of my commits. I will fix them today.
> > 1. Remove some unused codes and comments.
> > 2. fix UT_String_sprintf issues. (It is due to UT_String_vprintf
> > doesn't works well as my expected, I will read into the code to right
> > use it. )
> >
> > I will commit these changes soon! Thanks for your advices.
> >
> > Also, GTK implementation is in process. a lot of code in src/wp/xp/
> > can be reused while windows implementation.
> >
> >
> > Vincent
> >
> >
> > 2012/7/10 Kathiravelu Pradeeban <kk.pradeeban@gmail.com>:
> >>
> >> On Wed, Jul 4, 2012 at 8:18 PM, <cvs@abisource.com> wrote:
> >>>
> >>> Author: vincent
> >>> Date: 2012-07-04 16:48:54 +0200 (Wed, 04 Jul 2012)
> >>> New Revision: 31463
> >>>
> >>> Modified:
> >>> abiword/branches/gsoc2012dialogs/src/wp/ap/win/ap_Win32Dialog_FormatFrame.cpp
> >>> abiword/branches/gsoc2012dialogs/src/wp/ap/xp/ap_Dialog_FormatFrame.cpp
> >>> Log:
> >>> Finish Code refactor for ap_Win32Dialog_FormatFrame
> >>> 1. Comment un-used code
> >>
> >> I notice, you have included some code segment, and later commented
> >> them out as they are unused. Pls remove them altogether from the code
> >> base if they are not used. Unused and commented out code segments will
> >> make the trunk untidy, when/once they are merged.
> >>
> >>> 2. Update height & width when user change them in UI in time.
> >>
> >> Commits are cheap. Feel free to commit each of the change, in its own commit.
> >> It will make it easy to
> >> 1) read and understand the commit.
> >> 2) revert the commit, if needed.
> >>
> >>> 3. using sprintf instead of UT_String_sprintf (I don't know why UT_String_vprintf can't convert correctly)
> >>
> >> Were you able to attend this? I might have missed a later commit - will check.
> >> As I said, you could have made these changes in 3 different commits.
> >>
> >> Thank you.
> >> Regards,
> >> Pradeeban.
> >>
> >>>
> >>> Modified: abiword/branches/gsoc2012dialogs/src/wp/ap/win/ap_Win32Dialog_FormatFrame.cpp
> >>> ===================================================================
> >>> --- abiword/branches/gsoc2012dialogs/src/wp/ap/win/ap_Win32Dialog_FormatFrame.cpp 2012-07-03 23:37:22 UTC (rev 31462)
> >>> +++ abiword/branches/gsoc2012dialogs/src/wp/ap/win/ap_Win32Dialog_FormatFrame.cpp 2012-07-04 14:48:54 UTC (rev 31463)
> >>> @@ -197,11 +197,11 @@
> >>> /* Set the value of TEXT BOX */
> >>> wchar_t szValue[BUFSIZE];
> >>> //init value is current Frame width and height
> >>> - FV_View * pView = static_cast<FV_View *>(m_pApp->getLastFocussedFrame()->getCurrentView());
> >>> + /*FV_View * pView = static_cast<FV_View *>(m_pApp->getLastFocussedFrame()->getCurrentView());
> >>> if (!pView)
> >>> return 1;
> >>> fl_FrameLayout * pFL = pView->getFrameLayout();
> >>> -
> >>> + */
> >>> setCurFrameProps();
> >>> swprintf(szValue, L"%02.2f", getFrameWidth());
> >>> SetDlgItemTextW(m_hDlg, AP_RID_DIALOG_FORMATFRAME_VAL_WIDTH, szValue);
> >>> @@ -375,8 +375,8 @@
> >>> {
> >>> UT_LocaleTransactor t(LC_NUMERIC, "C");
> >>> UT_Win32LocaleString thickness;
> >>> + getComboTextItem(AP_RID_DIALOG_FORMATFRAME_COMBO_THICKNESS, nSelected, thickness);
> >>> UT_UTF8String thickness_utf8 = thickness.utf8_str ();
> >>> - getComboTextItem(AP_RID_DIALOG_FORMATFRAME_COMBO_THICKNESS, nSelected, thickness);
> >>> setBorderThicknessAll(thickness_utf8);
> >>> event_previewExposed();
> >>> }
> >>> @@ -462,16 +462,16 @@
> >>> CheckDlgButton(m_hDlg, AP_RID_DIALOG_FORMATFRAME_CHK_TEXTWRAP, getWrapping()? BST_CHECKED: BST_UNCHECKED);
> >>> //update height and width
> >>> /* FIXME: if update in the way, will cause the users can't input because it update all the time
> >>> - if((HWND)GetWindowLongPtrW(m_hDlg, GWLP_HWNDPARENT) == static_cast<XAP_Win32FrameImpl*>(getActiveFrame()->getFrameImpl())->getTopLevelWindow())
> >>> - {
> >>> - wchar_t szValue[BUFSIZE];
> >>> - swprintf(szValue, L"%02.2f", getFrameWidth());
> >>> - SetDlgItemTextW(m_hDlg, AP_RID_DIALOG_FORMATFRAME_VAL_WIDTH, szValue);
> >>> + */
> >>> + // Reset the value using the value from Prop vectot
> >>> + initFrameWidthStr();
> >>> + initFrameHeightStr();
> >>> + wchar_t szValue[BUFSIZE];
> >>> + swprintf(szValue, L"%02.2f", getFrameWidth());
> >>> + SetDlgItemTextW(m_hDlg, AP_RID_DIALOG_FORMATFRAME_VAL_WIDTH, szValue);
> >>>
> >>> - swprintf(szValue, L"%02.2f", getFrameHeight());
> >>> - SetDlgItemTextW(m_hDlg, AP_RID_DIALOG_FORMATFRAME_VAL_HEIGHT, szValue);
> >>> - }
> >>> - */
> >>> + swprintf(szValue, L"%02.2f", getFrameHeight());
> >>> + SetDlgItemTextW(m_hDlg, AP_RID_DIALOG_FORMATFRAME_VAL_HEIGHT, szValue);
> >>> }
> >>>
> >>> void AP_Win32Dialog_FormatFrame::destroy(void)
> >>>
> >>> Modified: abiword/branches/gsoc2012dialogs/src/wp/ap/xp/ap_Dialog_FormatFrame.cpp
> >>> ===================================================================
> >>> --- abiword/branches/gsoc2012dialogs/src/wp/ap/xp/ap_Dialog_FormatFrame.cpp 2012-07-03 23:37:22 UTC (rev 31462)
> >>> +++ abiword/branches/gsoc2012dialogs/src/wp/ap/xp/ap_Dialog_FormatFrame.cpp 2012-07-04 14:48:54 UTC (rev 31463)
> >>> @@ -946,7 +946,7 @@
> >>> else {
> >>> char buf[16];
> >>> UT_LocaleTransactor t(LC_NUMERIC, "C");
> >>> - UT_String_sprintf(buf, "%.2fpt", thickness);
> >>> + sprintf(buf, "%.2fpt", thickness);
> >>> sThick = buf;
> >>> }
> >>> return sThick;
> >>> @@ -969,7 +969,7 @@
> >>> else {
> >>> char buf[16];
> >>> UT_LocaleTransactor t(LC_NUMERIC, "C");
> >>> - UT_String_sprintf(buf, "%.2fpt", thickness);
> >>> + sprintf(buf, "%.2fpt", thickness);
> >>> sThick = buf;
> >>> }
> >>> return sThick;
> >>> @@ -988,7 +988,7 @@
> >>> else {
> >>> char buf[16];
> >>> UT_LocaleTransactor t(LC_NUMERIC, "C");
> >>> - UT_String_sprintf(buf, "%.2fpt", height_width);
> >>> + sprintf(buf, "%.2fpt", height_width);
> >>> sHeight_width = buf;
> >>> }
> >>> return sHeight_width;
> >>> @@ -1011,7 +1011,7 @@
> >>> else {
> >>> char buf[16];
> >>> UT_LocaleTransactor t(LC_NUMERIC, "C");
> >>> - UT_String_sprintf(buf, "%.2fpt", height_width);
> >>> + sprintf(buf, "%.2fpt", height_width);
> >>> sHeight_width_new = buf;
> >>> }
> >>> return sHeight_width_new;
> >>>
> >>> -----------------------------------------------
> >>> To unsubscribe from this list, send a message to
> >>> abisource-cvs-commit-request@abisource.com with the word
> >>> unsubscribe in the message body.
> >>
> >>
> >>
> >> --
> >> Kathiravelu Pradeeban.
> >> Blog: [Llovizna] http://kkpradeeban.blogspot.com/
>
>
>
> --
> Kathiravelu Pradeeban.
> Blog: [Llovizna] http://kkpradeeban.blogspot.com/
Received on Tue Jul 10 12:21:51 2012
This archive was generated by hypermail 2.1.8 : Tue Jul 10 2012 - 12:21:52 CEST