Issue 80635 - conversion return type from char* to ByteString
Summary: conversion return type from char* to ByteString
Status: CONFIRMED
Alias: None
Product: utilities
Classification: Unclassified
Component: code (show other issues)
Version: 680m224
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 3.x
Assignee: AOO issues mailing list
QA Contact: issues@tools
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-08-13 08:26 UTC by jnavrati
Modified: 2017-05-20 11:28 UTC (History)
1 user (show)

See Also:
Issue Type: PATCH
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
patch that convert char* to ByteString (5.09 KB, text/plain)
2007-08-13 08:27 UTC, jnavrati 		no flags Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description jnavrati 2007-08-13 08:26:12 UTC 
In a few methods that read a environment variables I've changed a return type
from char* to ByteString. There are a few reasons, first we allocated memory
using malloc that wasn't released in some cases, and second, usually when the
methods were called, char* was passed as argument to ByteString constructor anyway.
Comment 1 jnavrati 2007-08-13 08:27:02 UTC 
Created attachment 47503 [details]
patch that convert char* to ByteString
Comment 2 carsten.driesner 2007-08-13 10:15:11 UTC 
cd->jnavrati: Thanks for your patch. I will check your patch as soon as
possible. May be this can last a little bit longer as I will be in vacation for
2.5 weeks.
Comment 3 carsten.driesner 2007-08-15 09:24:40 UTC 
cd->obo: As discussed please take over.

cd->jnavrati: Your patch improves code which should be moved to another project
as it's not used in the tools project anymore. Oliver (obo) will take over and
check your patch.
Comment 4 oliver.bolte 2007-12-03 15:30:32 UTC 
Due to work load setting target to OOo 3.0
Comment 5 oliver.bolte 2008-03-11 11:50:10 UTC 
@kz: AFAIK you'll clean up tool/bootstrp soon. Therefore I reassign this issue
to you.
Comment 6 kurt.zenker 2008-07-29 16:31:43 UTC 
set target to 3.x. With i80690 tools will be cleaned up, maybe the task will be
obsolate then.
Comment 7 Rob Weir 2013-03-11 15:00:01 UTC 
I'm adding this comment to all open issues with Issue Type == PATCH.  We have 220 such issues, many of them quite old.  I apologize for that.  

We need your help in prioritizing which patches should be integrated into our next release, Apache OpenOffice 4.0.

If you have submitted a patch and think it is applicable for AOO 4.0, please respond with a comment to let us know.

On the other hand, if the patch is no longer relevant, please let us know that as well.

If you have any general questions or want to discuss this further, please send a note to our dev mailing list:  dev@openoffice.apache.org

Thanks!

-Rob
Comment 8 Marcus 2017-05-20 11:28:07 UTC 
Reset assigne to the default "issues@openoffice.apache.org".