The MouseButtonDown and MouseMove methods in http://opengrok.libreoffice.org/xref/core/sd/source/ui/func/fudraw.cxx share a lot of code.
This needs to be cleaned up and the shared code put in a method which both call (this file has the same methods and can serve as reference: http://opengrok.libreoffice.org/xref/core/sc/source/ui/drawfunc/fudraw.cxx).
I'll work on this. It'll be my first contribution, but the work to be done here looks pretty simple, so i guess i won't have many problems.
Created attachment 108854 [details]
git diff of the fudraw.cxx file at sd/source/ui/func
Sorry for the double comment... forgot to write the message.
So, sorry for taking so long to do something, but here it is.
I'm having problems in setting up gerrit. I hope i get it working by the end of the day, but i uploaded the git diff of the fudraw.cxx file, so you can say if it is possible to move more things do the DoModifiers and things like that.
I thought i would be able to move a lot of things to the new function, like in the fudraw file Samuel passed as a reference, but there were some differences in the code of both functions, although very similar.
About gerrit, i'm following the tutorial to set it up, but when i try ./logerrit test, it says it has a problem, and when i run ssh -vvvv logerrit, i get the message: "Could not resolve hostname logerrit: Name or service not known"
I also tryed to run "ssh -vvvv gerrit.libreoffice.org", but then i get "Permission denied (publickey)." And yes, i created an account at gerrit and put my public key there. It is the same key i use for github...so i'm kinda lost.
Well, i'll keep trying guys. If you could already give me some feedback, it would be great.
thanks for the patch!
Can you please make sure to have a license statement listed here: https://wiki.documentfoundation.org/Development/Developers
For help with gerrit, you might ask in IRC.
Hi! I was able to solve my gerrit problem by my self, and already pushed my commit for review:
I think i didn't commit right. If i understood right, usually there is a automatic comment in the bug's post when a person pushes a commit for a bug. Well, i'll try to get it right next time. Now i'll wait for the review.
(In reply to Victor Sanches Portella from comment #5)
> I think i didn't commit right. If i understood right, usually there is a
> automatic comment in the bug's post when a person pushes a commit for a bug.
The commit notification is triggered when the patch passes review.
Victor Portella committed a patch related to this issue.
It has been pushed to "master":
fdo#84380 Remove repeated mouse handling code
It will be available in 4.4.0.
The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
Affected users are encouraged to test the fix and report feedback.
Migrating Whiteboard tags to Keywords: (EasyHack DifficultyBeginner SkillCpp )