[ 
https://issues.apache.org/jira/browse/FLEX-33987?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13844738#comment-13844738
 ] 

Alexander Mazuruk commented on FLEX-33987:
------------------------------------------

I'm raising it as critical because it has more meaning to me & my project than 
to you.

I don't see how this npe check will slow down the framework as well I don't see 
it going bigger because of more npe cheks.
I don't see how you save my time with "fast" & "small" sdk code if I had to 
monkey patch every thing myself, doing it as rsl and loading before any stuff, 
this way you save my time & my efforts?  right?

Our project is huge financial business application, I cannot provide you with 
the exact use case  how to reproduce it, I've got this trace stack from the end 
users. I don't have resources and ability to spend some time to define exact 
steps for you to convince to fix this.

I've got an issue marked as critical from business application, because it does 
matter to the business users, when you've got npe in your code, further 
application functioning is totally unpredictable,  it's fast and compact, and 
with all that - it's uncontrolled and unpredictable, this is not what business 
users want, you see why it's critical?

Feel free to downgrade it to trivial as you wish, feel free not to fix it 
ether, do whatever you want, I just really have to get rid of those monkey 
patches growed through the years in our project. I need some cooperation from 
you.

> FocusManager NPE
> ----------------
>
>                 Key: FLEX-33987
>                 URL: https://issues.apache.org/jira/browse/FLEX-33987
>             Project: Apache Flex
>          Issue Type: Bug
>    Affects Versions: Apache Flex 4.11.0
>            Reporter: Alexander Mazuruk
>            Priority: Critical
>
> FocusManager Null pointer exception occurs.
> This:
>     public function getNextFocusManagerComponent(
>                             backward:Boolean = false):IFocusManagerComponent
>       {
>               return getNextFocusManagerComponent2(backward, 
> fauxFocus).displayObject as IFocusManagerComponent;
>       }
> Must be rewritten as:
> public function getNextFocusManagerComponent(
>                             backward:Boolean = false):IFocusManagerComponent
>       {
>         const focusInfo:FocusInfo = getNextFocusManagerComponent2(backward, 
> fauxFocus);
>         return focusInfo ? focusInfo.displayObject as IFocusManagerComponent 
> : null;
>       }



--
This message was sent by Atlassian JIRA
(v6.1.4#6159)

Reply via email to