Re: [Libreoffice] [PATCH] [PUSHED] Translate german comments of writer/sw/source/ui/cctrl
On 02/22/2011 12:06 AM, Martin Kepplinger wrote: Am 21.02.2011 23:08, schrieb Thomas Arnhold: Hi Martin, I've pushed it. Please could you attach such a diff as patch next time? Would be easier to commit it :) Thanks! Thomas On 02/21/2011 10:28 PM, Martin Kepplinger wrote: This translates the few german code comments in the cctrl directory to english. This is contributed under the terms of the MPL 1.1 / GPLv3+ / LGPLv3+ triple license. --- Hi Thomas, So no git format-patch and send-email anymore? I'm so used to it. I'm not a git pro, but I think you actually don't have to _commit_ any changes. I directly send git-commits here and you can preserve my sha1. I'm not sure if you did that in my case. (strangely, I find my patches two times under 2 different git-commits in master now... well, I'm not a git-Pro :) I would appreciate it if you could check with others (Christina Rossmanith for example pushed earlier patches of mine) about how they apply a finished git-patch and get back to me if I really should change my behaviour. Of course I will happily try to do so if this is consensus here! Since I thought I would make life easier for me _and_ you the git way, I just wanted to check once again here. Hi Martin, sorry I got this wrong. I applied one using 'git am' (c5fdfa6f475fc084250). But wasn't sure if all this text in the commit message should be there. So I manually applied the other two of you. Yes, send-email is welcome, didn't know that it was your intention with the commit message :) The sums are different, because for dialog stuff I changed it (as described). And for the other one, afaik I removed an additional comment. Thomas ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: [Libreoffice] [PATCH] [PUSHED] Translate german comments of writer/sw/source/ui/cctrl
On 02/22/2011 11:38 AM, Thomas Arnhold wrote: On 02/22/2011 12:06 AM, Martin Kepplinger wrote: Am 21.02.2011 23:08, schrieb Thomas Arnhold: Hi Martin, I've pushed it. Please could you attach such a diff as patch next time? Would be easier to commit it :) Thanks! Thomas On 02/21/2011 10:28 PM, Martin Kepplinger wrote: This translates the few german code comments in the cctrl directory to english. This is contributed under the terms of the MPL 1.1 / GPLv3+ / LGPLv3+ triple license. --- Hi Thomas, So no git format-patch and send-email anymore? I'm so used to it. I'm not a git pro, but I think you actually don't have to _commit_ any changes. I directly send git-commits here and you can preserve my sha1. I'm not sure if you did that in my case. (strangely, I find my patches two times under 2 different git-commits in master now... well, I'm not a git-Pro :) I would appreciate it if you could check with others (Christina Rossmanith for example pushed earlier patches of mine) about how they apply a finished git-patch and get back to me if I really should change my behaviour. Of course I will happily try to do so if this is consensus here! Since I thought I would make life easier for me _and_ you the git way, I just wanted to check once again here. Hi Martin, sorry I got this wrong. I applied one using 'git am' (c5fdfa6f475fc084250). But wasn't sure if all this text in the commit message should be there. So I manually applied the other two of you. Yes, send-email is welcome, didn't know that it was your intention with the commit message :) The sums are different, because for dialog stuff I changed it (as described). And for the other one, afaik I removed an additional comment. Mixed up something. I left your patches unchanged. So why they differ in the checksum I don't know. Thomas ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: [Libreoffice] [PATCH] [PUSHED] Translate german comments of writer/sw/source/ui/cctrl
Hi Martin, I've pushed it. Please could you attach such a diff as patch next time? Would be easier to commit it :) Thanks! Thomas On 02/21/2011 10:28 PM, Martin Kepplinger wrote: This translates the few german code comments in the cctrl directory to english. This is contributed under the terms of the MPL 1.1 / GPLv3+ / LGPLv3+ triple license. --- sw/source/ui/cctrl/popbox.cxx |2 +- sw/source/ui/cctrl/swlbox.cxx |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/sw/source/ui/cctrl/popbox.cxx b/sw/source/ui/cctrl/popbox.cxx index 9005eb3..c9f8a2b 100644 --- a/sw/source/ui/cctrl/popbox.cxx +++ b/sw/source/ui/cctrl/popbox.cxx @@ -44,7 +44,7 @@ SwHelpToolBox::SwHelpToolBox( SwNavigationPI* pParent, const ResId rResId ) void SwHelpToolBox::MouseButtonDown(const MouseEvent rEvt) { -// If doubleclick is detected use doublelick handler +// If doubleclick is detected use doubleclick handler if(rEvt.GetButtons() == MOUSE_RIGHT 0 == GetItemId(rEvt.GetPosPixel())) { diff --git a/sw/source/ui/cctrl/swlbox.cxx b/sw/source/ui/cctrl/swlbox.cxx index 908007f..3dd0c00 100644 --- a/sw/source/ui/cctrl/swlbox.cxx +++ b/sw/source/ui/cctrl/swlbox.cxx @@ -68,7 +68,7 @@ SwComboBox::SwComboBox(Window* pParent, const ResId rId, USHORT nStyleBits ): ComboBox(pParent, rId), nStyle(nStyleBits) { -// Verwaltung fuer die Stringlist aus der Resource aufbauen +// create administration for the resource's Stringlist USHORT nSize = GetEntryCount(); for( USHORT i=0; i nSize; ++i ) { ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: [Libreoffice] [PATCH] [PUSHED] Translate german comments of writer/sw/source/ui/cctrl
Am 21.02.2011 23:08, schrieb Thomas Arnhold: Hi Martin, I've pushed it. Please could you attach such a diff as patch next time? Would be easier to commit it :) Thanks! Thomas On 02/21/2011 10:28 PM, Martin Kepplinger wrote: This translates the few german code comments in the cctrl directory to english. This is contributed under the terms of the MPL 1.1 / GPLv3+ / LGPLv3+ triple license. --- Hi Thomas, So no git format-patch and send-email anymore? I'm so used to it. I'm not a git pro, but I think you actually don't have to _commit_ any changes. I directly send git-commits here and you can preserve my sha1. I'm not sure if you did that in my case. (strangely, I find my patches two times under 2 different git-commits in master now... well, I'm not a git-Pro :) I would appreciate it if you could check with others (Christina Rossmanith for example pushed earlier patches of mine) about how they apply a finished git-patch and get back to me if I really should change my behaviour. Of course I will happily try to do so if this is consensus here! Since I thought I would make life easier for me _and_ you the git way, I just wanted to check once again here. thanks! Martin -- /***\ | Martin Kepplinger | | | | diaspora handle martinkepplin...@joindiaspora.com | | | | E-Mail IM(jabber) martinkepplin...@eml.cc | | | | GPG Fingerprint 7346 FE02 27C3 3958 813A C351 C100 D7B5 7F2A 1E26 | \***/ ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: [Libreoffice] [PATCH] [PUSHED] Translate german comments of writer/sw/source/ui/cctrl
On Tue, 2011-02-22 at 00:06 +0100, Martin Kepplinger wrote: So no git format-patch and send-email anymore? I'm so used to it. No no, git format-patch is still the preferred patch format, so you are fine. For those who integrate patches, you can integrate patches generated by git format-patch simply by running 'git am' command. That's the preferred way because git format-patch preserves the commit message and the committer information as well as it handles changes in binary blobs. These are something the conventional patch format does not handle. Kohei -- Kohei Yoshida, LibreOffice hacker, Calc kyosh...@novell.com ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice