Bug 36234 - Easy Hack: Removes all bogus comments in WRITER repo
Summary: Easy Hack: Removes all bogus comments in WRITER repo
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: contrib (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: All All
: medium normal
Assignee: Júlio Hoffimann
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-14 06:28 UTC by Júlio Hoffimann
Modified: 2011-04-22 07:14 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:

Attachments
Removes all bogus comments in Writer repo (476.47 KB, patch)
2011-04-14 06:29 UTC, Júlio Hoffimann 		Details
All bogus comments removed in Writer repo (423.91 KB, patch)
2011-04-16 19:00 UTC, Júlio Hoffimann 		Details
Removes the remaining left arrow comments // <-- (463.10 KB, patch)
2011-04-22 04:26 UTC, Júlio Hoffimann 		Details
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Júlio Hoffimann 2011-04-14 06:28:13 UTC 
All bogus comments removed in Writer repo.
Comment 1 Júlio Hoffimann 2011-04-14 06:29:54 UTC 
Created attachment 45609 [details]
Removes all bogus comments in Writer repo
Comment 2 Rainer Bielefeld Retired 2011-04-14 10:41:02 UTC 
I also saw activities concerning "bogus comments":
Bug 34046 - Account request
Bug 30593 - Comments adjusted as requested in the wiki


@Cédric:
May be you can lead this to the correct place?
Please feel free to reassign if it's not your area!
Comment 3 Cédric Bosdonnat 2011-04-15 00:27:44 UTC 
Let's close the bug as the patch has been integrated... though bogus comments will reappear sooner or later with DEV300 merges. I already saw some in sw.

@Rainer: let's close bugs when these are applied patches.
Comment 4 Christian Lohmaier 2011-04-15 02:43:53 UTC 
closing, in order to make queries for QA more sensible
Comment 5 Cédric Bosdonnat 2011-04-15 03:24:39 UTC 
Reopening it: found that the patch wasn't applied and contained too many things to manually adjust to push it. See answer on mailing-list for more details.
http://lists.freedesktop.org/archives/libreoffice/2011-April/010541.html
Comment 6 Cédric Bosdonnat 2011-04-15 03:25:17 UTC 
Assigned the issue to Julio for the patches modifications.
Comment 7 Júlio Hoffimann 2011-04-16 19:00:05 UTC 
Created attachment 45721 [details]
All bogus comments removed in Writer repo

The bogus comments now were discarded keeping the indentation, bug id and the comment remainder resident at same line of the regexp
Comment 8 Cédric Bosdonnat 2011-04-18 07:48:42 UTC 
Patch applied, many thanks for the updates.
Comment 9 Júlio Hoffimann 2011-04-22 04:26:57 UTC 
Created attachment 45945 [details]
Removes the remaining left arrow comments // <--
Comment 10 Katarina Behrens 2011-04-22 07:14:23 UTC 
Attachment from comment #9 pushed. The remaining patches sent to ML too. We can close this one