[Libreoffice-bugs] [Bug 49602] Remove deprecated OSL Profile functionality

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=49602

khandelwalarvin...@gmail.com changed:

   What|Removed |Added

   Assignee|libreoffice-b...@lists.free |khandelwalarvin...@gmail.co
   |desktop.org |m

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 49602] Remove deprecated OSL Profile functionality

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=49602

BogdanB  changed:

   What|Removed |Added

 Blocks||143781


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=143781
[Bug 143781] [META] Development- and code-related bug reports and tasks
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 49602] Remove deprecated OSL Profile functionality

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=49602

Hossein  changed:

   What|Removed |Added

 Whiteboard||reviewed:2022

--- Comment #9 from Hossein  ---
Re-evaluating the EasyHack in 2022

This enhancement is still relevant. One should read comment 2 and follow the
steps written by Stephan.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 49602] Remove deprecated OSL Profile functionality

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=49602

Tor Lillqvist  changed:

   What|Removed |Added

   Assignee|alfredocol...@gmail.com |libreoffice-b...@lists.free
   ||desktop.org

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 49602] Remove deprecated OSL Profile functionality

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=49602

Alfredo  changed:

   What|Removed |Added

   Assignee|libreoffice-b...@lists.free |alfredocol...@gmail.com
   |desktop.org |

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 49602] Remove deprecated OSL Profile functionality

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=49602

jan iversen  changed:

   What|Removed |Added

 Whiteboard||ToBeReviewed

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 49602] Remove deprecated OSL Profile functionality

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=49602

Stephan Bergmann  changed:

   What|Removed |Added

 CC||libreoffice@lists.freedeskt
   ||op.org
 Whiteboard||EasyHack
   ||DifficultyInteresting
   ||SkillCpp TopicCleanup
   Severity|normal  |enhancement

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 49602] Remove deprecated OSL Profile functionality

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=49602

--- Comment #4 from Sergey  ---
the issue is still present in the latest GIT

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 49602] Remove deprecated OSL Profile functionality

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=49602

--- Comment #3 from QA Administrators qa-ad...@libreoffice.org ---
** Please read this message in its entirety before responding **

To make sure we're focusing on the bugs that affect our users today,
LibreOffice QA is asking bug reporters and confirmers to retest open, confirmed
bugs which have not been touched for over a year.

There have been thousands of bug fixes and commits since anyone checked on this
bug report. During that time, it's possible that the bug has been fixed, or the
details of the problem have changed. We'd really appreciate your help in
getting confirmation that the bug is still present.

If you have time, please do the following:

   *Test to see if the bug is still present on a currently supported version of
LibreOffice (4.4.1 or later)
   https://www.libreoffice.org/download/

   *If the bug is present, please leave a comment that includes the version of
LibreOffice and your operating system, and any changes you see in the bug
behavior

   *If the bug is NOT present, please set the bug's Status field to
RESOLVED-WORKSFORME and leave a short comment that includes your version of
LibreOffice and Operating System

Please DO NOT

   *Update the version field
   *Reply via email (please reply directly on the bug tracker)
   *Set the bug's Status field to RESOLVED - FIXED (this status has a
particular meaning that is not appropriate in this case)


If you want to do more to help you can test to see if your issue is a
REGRESSION. To do so: 

1. Download and install oldest version of LibreOffice (usually 3.3 unless your
bug pertains to a feature added after 3.3)

http://downloadarchive.documentfoundation.org/libreoffice/old/

2. Test your bug 
3. Leave a comment with your results. 
4a. If the bug was present with 3.3 - set version to inherited from OOo; 
4b. If the bug was not present in 3.3 - add regression to keyword


Feel free to come ask questions or to say hello in our QA chat:
http://webchat.freenode.net/?channels=libreoffice-qa

Thank you for your help!

-- The LibreOffice QA Team This NEW Message was generated on: 2015-04-01

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 49602] Remove deprecated OSL Profile functionality

[bugzilla-daemon]

https://bugs.freedesktop.org/show_bug.cgi?id=49602

Stephan Bergmann sberg...@redhat.com changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Whiteboard|Need_Advice |
Summary|Remove deprecated   |Remove deprecated OSL
   |include/osl/profile.hxx |Profile functionality
 Ever confirmed|0   |1

--- Comment #2 from Stephan Bergmann sberg...@redhat.com ---
(In reply to comment #0)
 Seems the only user of it is sal/rtl/source/bootstrap.cxx
 
 it could easily be replaced with something like:
 
 oslProfile profile = osl_openProfile(strProfileName.pData, Options);
 if( ! profile )
 throw std::exception();
 
 sal_Char aBuf[1024];
 rtl::OString ret= osl_readProfileString( profile, rSection.getStr(),
 rEntry.getStr(), aBuf, sizeof( aBuf ), rDefault.getStr() ) ? rtl::OString(
 aBuf ) : rtl::OString();
 
 osl_closeProfile(profile);
 
 return ret;

Given the comment preceding the (only) use of osl::Profile in
sal/rtl/bootstrap.cxx,

  // Going through osl::Profile, this code erroneously
  // does not recursively expand macros in the resulting
  // replacement text (and if it did, it would fail to
  // detect cycles that pass through here):

it does not make much sense to merely unwrap that code from using the C++
osl/profile.hxx wrapper to using the underlying C osl/profile.h functionality.

Also note that this is part of the stable URE interface, so cannot be modified
arbitrarily (but can arguably be removed in an incompatible release, as it is
deprecated for a long time and should effectively be unused by client code).

So, a useful approach could be:

* Adapt sal/rtl/bootstrap.cxx to no longer use the OSL Profile functionality,
and in doing so fix the bug described in the comment above.

* In a commit with [API CHANGE] in the commit message, remove the
include/osl/profile.hxx wrapper.

* In the same or a subsequent [API CHANGE] commit, also remove
include/sal/profile.h and replace the implementation with stubs in
sal/osl/all/compat.cxx (which are necessary as the corresponding symbols could
not be removed from sal/util/sal.map without changing the sal SONAME, which we
do not want to do).

* Document the incompatible changes in the API Changes section of the
relevant https://wiki.documentfoundation.org/ReleaseNotes.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs