Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Trivial 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
        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
        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:
            Martin Kočí
            Reporter:
            Martin Kočí
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development