| Summary: | Tangent is not recognize | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Dorange-Pattoret Didier <ddorange> |
| Component: | BASIC | Assignee: | Laurent Balland <jumbo4444> |
| Status: | VERIFIED FIXED | ||
| Severity: | normal | CC: | cno, jbfaure, jumbo4444, rb.henschel, xiscofauli |
| Priority: | medium | Keywords: | bibisected, bisected, regression |
| Version: | 5.2.0.0.alpha0+ | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | target:5.3.0 target:5.2.3 | ||
| Crash report or crash signature: | Regression By: | ||
|
Description
Dorange-Pattoret Didier
2016-09-30 15:01:21 UTC
Reproducible with master and version 5.2.3.0+ under Ubuntu x86-64. Confirmed on W7 on French users ML. -> status NEW. Best regards. JBF ok in 5.1.5 indeed. Thanks for reporting! Suspected commit: 98f0e0eda50690bd01842c806bd13e3bc5af2966 https://cgit.freedesktop.org/libreoffice/core/commit/basic/?id=98f0e0eda50690bd01842c806bd13e3bc5af2966&context=8&ignorews=0&ss=0 Declaration of TAB function is missing its argument. So it uses TAN function as argument in basic/source/runtime/stdobj.cxx http://opengrok.libreoffice.org/xref/core/basic/source/runtime/stdobj.cxx#606 Hi Laurent, Thanks for pointing out the commit that introduced the regression. I've reverted it locally and it's indeed the problematic one. Adding: bisected and bibisected to the keyword. Would you like to fix this? (In reply to Xisco Faulí from comment #4) > Would you like to fix this? I've proposed a commit https://gerrit.libreoffice.org/#/c/29437/ Laurent Balland-Poirier committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=db29cdcc70ba792ed54e0957929b4524816b1fb3 tdf#102872 TAB function is missing its argument It will be available in 5.3.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: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Laurent Balland-Poirier committed a patch related to this issue. It has been pushed to "libreoffice-5-2": http://cgit.freedesktop.org/libreoffice/core/commit/?id=74817e67441204416a4e6db0440d35c85a2fca84&h=libreoffice-5-2 tdf#102872 TAB function is missing its argument It will be available in 5.2.3. 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: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Hi Laurent, Thanks for your fix. You can close the bug as RESOLVED FIXED now. Thank you very much for this fix. It works as expected in Version: 5.2.3.0.0+ Build ID: 3b0a57d6de5931fcbd554a11cb9abdd09c34a735 Threads CPU : 4; Version de l'OS :Linux 4.4; UI Render : par défaut; Ubuntu_16.04_x86-64 Locale : fr-FR (fr_FR.UTF-8); Calc: single Closing as Resolved Fixed then Verified Fixed. Best regards. JBF Laurent Balland-Poirier committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=8a50ea54546a51fc7a2a28174031e7351dbed10e tdf#102872 Add qa unit test for BASIC methods It will be available in 5.3.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: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. |