Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.13, 2.1.7
    • Component/s: JSR-314
    • Labels:
      None
    • Environment:
      myfaces core trunk

      Description

      see MYFACES-3130 for details

      1. FaceletViewDeclarationLanguage.patch
        2 kB
        Martin Kočí
      2. UIViewRoot.patch
        4 kB
        Martin Kočí
      3. UIData.patch
        0.9 kB
        Martin Kočí
      4. UIComponentBase.patch
        3 kB
        Martin Kočí
      5. RenderResponseExecutor.patch
        2 kB
        Martin Kočí
      6. PartialVisitContext.patch
        3 kB
        Martin Kočí
      7. HtmlRendererUtils.patch
        1 kB
        Martin Kočí
      8. HtmlFormatRenderer.patch
        0.9 kB
        Martin Kočí
      9. HtmlAjaxBehaviorRenderer.patch
        3 kB
        Martin Kočí
      10. _DeltaList2.patch
        0.8 kB
        Martin Kočí
      11. _DeltaList.patch
        0.7 kB
        Martin Kočí
      12. _ComponentChildrenList.patch
        0.8 kB
        Martin Kočí
      13. ApplicationImpl.patch
        1 kB
        Martin Kočí

        Activity

        Hide
        lu4242 Leonardo Uribe added a comment -

        I checked the patches and I commit almost all. I let these ones out:

        • _DeltaList2 : that class will replace _DeltaList, but needs some checking first. I'll apply it later when the change is complete.
        • ApplicationImpl : An iterator is supposed to be used per thread unless it will be empty. The patch stores the iterator at application scope, so it could break on concurrency.
        • UIViewRoot : I only applied the optimization related to Events. I prefer let the other code with an iterator, because the alternative makes the code difficult to read, and that part was specially very difficult to tune.
        Show
        lu4242 Leonardo Uribe added a comment - I checked the patches and I commit almost all. I let these ones out: _DeltaList2 : that class will replace _DeltaList, but needs some checking first. I'll apply it later when the change is complete. ApplicationImpl : An iterator is supposed to be used per thread unless it will be empty. The patch stores the iterator at application scope, so it could break on concurrency. UIViewRoot : I only applied the optimization related to Events. I prefer let the other code with an iterator, because the alternative makes the code difficult to read, and that part was specially very difficult to tune.

          People

          • Assignee:
            markoc50 Martin Kočí
            Reporter:
            markoc50 Martin Kočí
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development