Changes between Initial Version and Version 1 of Ticket #7003, comment 4


Ignore:
Timestamp:
09/30/2018 07:52:09 AM (7 months ago)
Author:
mjethani
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #7003, comment 4

    initial v1  
    77There's also the option of keeping the current function as it is but adding a new `isBlocked` function that returns a `boolean` value, but this is just a variation of the second option above. 
    88 
    9 I personally thing we should go with option one. It would be simple enough and there would be no downside to it for the extension. Yes, it does mean that there would two iterations instead of one for blocked URLs, but given that most URLs aren't blocked, on average we should be using less CPU than before. 
     9I personally think we should go with option one. It would be simple enough and there would be no downside to it for the extension. Yes, it does mean that there would two iterations instead of one for blocked URLs, but given that most URLs aren't blocked, on average we should be using less CPU than before. 
    1010