This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
Summary: | For now, ExecInfo should not write the warning message to IDE log file | ||
---|---|---|---|
Product: | platform | Reporter: | Petr Jiricka <pjiricka> |
Component: | Execution | Assignee: | Jesse Glick <jglick> |
Status: | VERIFIED FIXED | ||
Severity: | blocker | CC: | jchalupa, jglick, mkuchtiak, mmirilovic, ttran |
Priority: | P1 | ||
Version: | 3.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: | |
Bug Depends on: | |||
Bug Blocks: | 32153 | ||
Attachments: |
Proposed source diff for r35
Binary patch for r35 - place in $nbhome/modules/autoload/patches/org-openide-execution/32589.jar |
Description
Petr Jiricka
2003-04-03 08:33:51 UTC
Do we really need it fixed for 3.5? If yes rise priority to P1, thanks ? Jesse, what is your opinion on this? > writing a message to IDE log is not the right way to inform about a
> deprecated contract in IDE code.
what is the right way then?
I don't have a strong opinion on it. ExecInfo is also @deprecated. The warning stack trace proved useful in tracking down places where it was being used from, but it could be removed and leave the compile-time deprecation. > > writing a message to IDE log is not the right way to
> > inform about a deprecated contract in IDE code.
>
> what is the right way then?
>
Sorry, I really meant to say that the final product should
not have such warnings. Dev builds may have them and
warnings are appropriate in dev builds.
Ok, QA (me and Honza) "recommend" to fix this issue for 3.5 release. so Jesse please remove the warning. You have my approval to get the fix in r35. Of course you still need to attach the diff/patch and file IN. Created attachment 9685 [details] Proposed source diff for r35 Created attachment 9686 [details] Binary patch for r35 - place in $nbhome/modules/autoload/patches/org-openide-execution/32589.jar diff reviewed, patch verified, fix approved for r35 --Trung committed Up-To-Date 1.4.34.1 openide/execution/src/org/openide/execution/ExecInfo.java verified. No message in log about deprecated execution. |