ICEfaces
  1. ICEfaces
  2. ICE-5573

remove unused ThreadBlocking classes

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Alpha2
    • Fix Version/s: 2.0-Beta1, 2.0.0
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      ICEfaces 2.0

      Description

      ICEfaces 2.0 now makes use of ICEpush, so does not require an internal implementation of thread blocking or other asynchronous features.

        Activity

        Hide
        Ted Goddard added a comment -

        These classes are Servlet-specific, so removal is useful preparation for Portlet compatibility work.

        Show
        Ted Goddard added a comment - These classes are Servlet-specific, so removal is useful preparation for Portlet compatibility work.
        Hide
        Ted Goddard added a comment -

        ICEfaces delegates all push functionality to ICEpush, so ICEfaces code should contain no push implementation. Only the PushRenderer and a JSF ResourceHandler for integrating with ICEpush should be present. Further examination of the codebase is required to determine if other unused classes (beyond the ones that have already been removed) should be removed.

        Show
        Ted Goddard added a comment - ICEfaces delegates all push functionality to ICEpush, so ICEfaces code should contain no push implementation. Only the PushRenderer and a JSF ResourceHandler for integrating with ICEpush should be present. Further examination of the codebase is required to determine if other unused classes (beyond the ones that have already been removed) should be removed.
        Hide
        Mircea Toma added a comment -

        After analyzing the 'core' project source code I cannot detect any use of thread blocking strategies (such as wait/notify, mutexes or semaphores).

        Show
        Mircea Toma added a comment - After analyzing the 'core' project source code I cannot detect any use of thread blocking strategies (such as wait/notify, mutexes or semaphores).

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Ted Goddard
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: