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

              Ascending order - Click to sort in descending order
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #21125 Wed Mar 31 13:33:34 MDT 2010 ted.goddard removed unused classes (ICE-5573)
              Files Changed
              Commit graph DEL /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/push/servlet/ThreadBlockingAdaptingServlet.java
              Commit graph DEL /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/push/servlet/EnvironmentAdaptingException.java
              Commit graph DEL /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/push/servlet/AsyncAdaptingServlet.java
              Commit graph DEL /icefaces/scratchpads/glimmer/core/src/main/java/org/icefaces/push/servlet/EnvironmentAdaptingServlet.java
              Ted Goddard created issue -
              Hide
              Permalink
              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.
              Ken Fyten made changes -
              Field Original Value New Value
              Salesforce Case []
              Fix Version/s 2.0-Beta [ 10231 ]
              Assignee Deryk Sinotte [ deryk.sinotte ]
              Ken Fyten made changes -
              Salesforce Case []
              Component/s Framework [ 10013 ]
              Ken Fyten made changes -
              Salesforce Case []
              Assignee Priority P1
              Hide
              Permalink
              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.
              Deryk Sinotte made changes -
              Assignee Deryk Sinotte [ deryk.sinotte ] Mircea Toma [ mircea.toma ]
              Hide
              Permalink
              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).
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 2.0.0 [ 10230 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              Assignee Priority P1

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: