https://bugs.documentfoundation.org/show_bug.cgi?id=139734
Mike Kaganski changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #33 from Commit Notification
---
Siddhant Chaudhary committed a patch related to this issue.
It has been pushed to "master":
https://git.libreoffice.org/core/commit/43f6ca7164a9ea7c99d177d711684e25d53e8be4
tdf#139734 Drop
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #32 from Mike Kaganski ---
(In reply to Hannah Meeks from comment #31)
Only one :-)
--
You are receiving this mail because:
You are the assignee for the bug.
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #31 from Hannah Meeks ---
Are there any more of these left?
--
You are receiving this mail because:
You are the assignee for the bug.
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #30 from Commit Notification
---
pragat-pandya committed a patch related to this issue.
It has been pushed to "master":
https://git.libreoffice.org/core/commit/73ef2f6f615b88ef25366f9073c5fb3db5e76687
tdf#139734 removing
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #29 from Mike Kaganski ---
(In reply to comment #28)
"This easy hack is about finding all the places where the method is used, and
removing the direct assert on its return value if present."
The idea is: remove all of them,
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #28 from Libreoffice user SSO ---
I'm currently working on this easyhack. And I came across some instances where
a call to CPPUNIT_ASSERT_MESSAGE was made checking mxComponent.is(). Isn't
CPPUNIT_ASSERT_MESSAGE is also
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
Hossein changed:
What|Removed |Added
Status|ASSIGNED|NEW
--
You are receiving this mail
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
pjrad...@gmail.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
--
You are
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #27 from Commit Notification
---
yalda committed a patch related to this issue.
It has been pushed to "master":
https://git.libreoffice.org/core/commit/e9fce801c2a888b76fd8230343b04f72b5b6b5ce
tdf#139734 deleting redundant
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
Commit Notification changed:
What|Removed |Added
Whiteboard|target:7.2.0 target:7.3.0 |target:7.2.0 target:7.3.0
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #26 from Commit Notification
---
Henrik Palomäki committed a patch related to this issue.
It has been pushed to "master":
https://git.libreoffice.org/core/commit/0935a53a1ef70d0cbaef5fbfe2a0bc2f3ddfeafd
tdf#139734 Drop
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #25 from Commit Notification
---
4k5h1t committed a patch related to this issue.
It has been pushed to "master":
https://git.libreoffice.org/core/commit/7970e700f86bca811508f9b229110d6ab4393d04
tdf#139734 Drop redundant
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #24 from Commit Notification
---
Radhey Parekh committed a patch related to this issue.
It has been pushed to "master":
https://git.libreoffice.org/core/commit/b3f3f0cb2a5ea7873349f733b55e41c1cbc671dc
tdf#139734 Drop
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #23 from Commit Notification
---
Yildiray committed a patch related to this issue.
It has been pushed to "master":
https://git.libreoffice.org/core/commit/08058894d2da9e65b91a1dcdb997b70f69448fb3
tdf#139734:Drop redundant
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #22 from Commit Notification
---
KaranAbrol committed a patch related to this issue.
It has been pushed to "master":
https://git.libreoffice.org/core/commit/c9bcbebe34c8d0e3edab520ace15c4d367aef62f
tdf#139734 Drop
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #21 from Commit Notification
---
Karan Abrol committed a patch related to this issue.
It has been pushed to "master":
https://git.libreoffice.org/core/commit/7b21c25354d16b9d9e04f36c1370b9a2edfe09fc
tdf#139734 Remove
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
Commit Notification changed:
What|Removed |Added
Whiteboard|target:7.2.0|target:7.2.0 target:7.3.0
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #20 from Mike Kaganski ---
(In reply to Karan Abrol from comment #19)
Hi!
Thanks for working on this.
I have resumed the build that apparently failed for an unrelated reason.
An advise/reminder to everyone: please add the
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #19 from Karan Abrol ---
Hey, I tried submitting my first patch
(https://gerrit.libreoffice.org/c/core/+/120319).
I ran make check successfully on my system but the build by Jenkins failed.
Could you please review it?
--
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
Aron Budea changed:
What|Removed |Added
Blocks||143781
Referenced Bugs:
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #18 from Mike Kaganski ---
(In reply to Sabyasachi Bhoi from comment #17)
> The CPPUNIT_ASSERT_MESSAGE Macros don't seem to throw an error when given a
> false value while testing the function from which loadFromDesktop() is
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #17 from Sabyasachi Bhoi ---
The CPPUNIT_ASSERT_MESSAGE Macros don't seem to throw an error when given a
false value while testing the function from which loadFromDesktop() is called.
Are these tests not supposed to run when
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #16 from Commit Notification
---
sarynasser committed a patch related to this issue.
It has been pushed to "master":
https://git.libreoffice.org/core/commit/8ed68179e2eaf07289663c64d923d8b5dd8b25c9
tdf#139734 remove
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #15 from Commit Notification
---
Ahmet Hakan Çelik committed a patch related to this issue.
It has been pushed to "master":
https://git.libreoffice.org/core/commit/c1d10276d32cbdaecf6894be723ab8d29cc7dbe2
tdf#139734: Drop
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #14 from Buovjaga ---
Forgot to note that you should be careful not to step on the toes of others by
looking at the currently unmerged patches:
https://gerrit.libreoffice.org/q/message:139734+status:open
--
You are
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
Buovjaga changed:
What|Removed |Added
CC||kumarashwin2...@gmail.com
---
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #12 from Ashwin ---
Hi,
I'm new to open source contributions.
This whole forums looks too cryptic to me, where can I find the code to try to
contribute to it?
Any advice for a newbie would be greatful.
Ashwin
--
You are
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #11 from Commit Notification
---
dipanshu124 committed a patch related to this issue.
It has been pushed to "master":
https://git.libreoffice.org/core/commit/993298cfff01e09142c98faf03c3dbb608369114
tdf#139734 Drop
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #10 from Commit Notification
---
Moaz committed a patch related to this issue.
It has been pushed to "master":
https://git.libreoffice.org/core/commit/dbbda04d52e75858840ce9748a19a61a1c565149
tdf#139734 Remove redundant
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #9 from Dipanshu Garg ---
Hi, I submitted a patch for this.
https://gerrit.libreoffice.org/c/core/+/111810
But the build by jenkins is failing, although make check was successful on my
system.
Can you please review it?
--
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #8 from Commit Notification
---
Moaz committed a patch related to this issue.
It has been pushed to "master":
https://git.libreoffice.org/core/commit/a9a350c13d8fb454c3f1db7ca6a7f10c1e7a19bb
tdf#139734 Remove redundant
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
Commit Notification changed:
What|Removed |Added
Whiteboard||target:7.2.0
--
You are
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #7 from Mike Kaganski ---
(In reply to Moaz El-defrawy from comment #6)
> > > I am not sure how style the comments to make them more readable. I tried
> > > html and readme files styling but neither worked.
> >
> > Are you
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #6 from Moaz El-defrawy ---
(In reply to Mike Kaganski from comment #5)
> (In reply to Moaz El-defrawy from comment #4)
> > Update: I am removed 99% of the redundant asserts and I will submit a patch
> > very soon.
>
> Very
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #5 from Mike Kaganski ---
(In reply to Moaz El-defrawy from comment #4)
> Update: I am removed 99% of the redundant asserts and I will submit a patch
> very soon.
Very good!
> I had a question though:
> There is a pattern
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #4 from Moaz El-defrawy ---
Update: I am removed 99% of the redundant asserts and I will submit a patch
very soon.
I had a question though:
There is a pattern of asserting the return values of many functions not just
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #3 from Mike Kaganski ---
(In reply to Moaz El-defrawy from comment #1)
> Hey, can I work on this?
Welcome! Of course, please do. Everyone is welcome; this task may be done by
multiple developers in parallel :-)
--
You are
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #2 from Moaz El-defrawy ---
(In reply to Mike Kaganski from comment #0)
> In many unit tests (e.g., in dbaccess/qa/extras/empty-stdlib-save.cxx), we
> have this pattern asserting on return value of
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
--- Comment #1 from Moaz El-defrawy ---
Hey, can I work on this?
--
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
Aron Budea changed:
What|Removed |Added
Ever confirmed|0 |1
CC|
https://bugs.documentfoundation.org/show_bug.cgi?id=139734
QA Administrators changed:
What|Removed |Added
Whiteboard|| QA:needsComment
--
You
42 matches
Mail list logo