Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[Bug] NREs still occur in XF #14204

Open
charlesroddie opened this issue Apr 22, 2021 · 2 comments
Open

[Bug] NREs still occur in XF #14204

charlesroddie opened this issue Apr 22, 2021 · 2 comments
Labels
i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often s/needs-info ❓ A question has been asked that requires an answer before work can continue on this issue. s/needs-repro ❔ This reported issue doesn't include a sample project reproducing the issue. Please provide one. s/unverified New report that has yet to be verified t/bug 🐛
Projects

Comments

@charlesroddie
Copy link

charlesroddie commented Apr 22, 2021

NREs still occur in XF, 2 YEARS after the solution (NRTs) was rolled out across .Net.
Serious mismanagement.

iOS crash report:

NavigableElement.OnParentSet ()
SIGABRT: Object reference not set to an instance of an object.

NavigableElement.OnParentSet ()
Element.set_Parent (Xamarin.Forms.Element value)
Element.OnChildAdded (Xamarin.Forms.Element child)
VisualElement.OnChildAdded (Xamarin.Forms.Element child)
Layout`1[T].OnChildAdded (Xamarin.Forms.Element child)
Layout.OnInternalAdded (Xamarin.Forms.View view)
Layout.InternalChildrenOnCollectionChanged (System.Object sender, System.Collections.Specialized.NotifyCollectionChangedEventArgs e)
(wrapper delegate-invoke) <Module>.invoke_void_object_NotifyCollectionChangedEventArgs(object,System.Collections.Specialized.NotifyCollectionChangedEventArgs)
ObservableCollection`1[T].OnCollectionChanged (System.Collections.Specialized.NotifyCollectionChangedEventArgs e)
ObservableCollection`1[T].OnCollectionChanged (System.Collections.Specialized.NotifyCollectionChangedAction action, System.Object item, System.Int32 index)
ObservableCollection`1[T].InsertItem (System.Int32 index, T item)
Collection`1[T].Add (T item)
ObservableWrapper`2[TTrack,TRestrict].Add (TRestrict item)
Grid+GridElementCollection.Add (Xamarin.Forms.View view, System.Int32 left, System.Int32 right, System.Int32 top, System.Int32 bottom)
Grid+GridElementCollection.Add (Xamarin.Forms.View view, System.Int32 left, System.Int32 top
@charlesroddie charlesroddie added s/unverified New report that has yet to be verified t/bug 🐛 labels Apr 22, 2021
@jsuarezruiz jsuarezruiz added the i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often label Apr 28, 2021
@jsuarezruiz jsuarezruiz added this to New in Triage via automation Apr 28, 2021
@rachelkang
Copy link
Contributor

Hi @charlesroddie - thanks for submitting this issue. Would you be able to share your project or a sample project reproducing the issue so that we can take a closer look? In what scenario(s) are you seeing these NREs thrown? Are they also causing crashes?

@rachelkang rachelkang moved this from New to Needs Info in Triage Apr 28, 2021
@rachelkang rachelkang added s/needs-info ❓ A question has been asked that requires an answer before work can continue on this issue. s/needs-repro ❔ This reported issue doesn't include a sample project reproducing the issue. Please provide one. labels Apr 28, 2021
@charlesroddie
Copy link
Author

charlesroddie commented May 9, 2021

@rachelkang thanks for asking and sorry for the very slow reply.

This is a crash report on iOS. It's very sporadic. Nothing that we have reproduced internally.

The cause is that NRTs are not enabled in the Xamarin.Forms repo so errors like this are expected.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often s/needs-info ❓ A question has been asked that requires an answer before work can continue on this issue. s/needs-repro ❔ This reported issue doesn't include a sample project reproducing the issue. Please provide one. s/unverified New report that has yet to be verified t/bug 🐛
Projects
Triage
  
Needs Info
Development

No branches or pull requests

3 participants