Since the code exists to handle the error I think the change is good.

Reviewed-by: Erik Bjorge <erik.c.bjo...@intel.com>

From: Carsey, Jaben
Sent: Monday, December 09, 2013 2:59 PM
To: Bjorge, Erik C
Cc: edk2-devel@lists.sourceforge.net; Carsey, Jaben
Subject: ShellPkg: Remove invalid ASSERT

Erik

Can you check this?

ShellPkg: Remove invalid ASSERT

There was an assumption that this API would never fail.  That is not true and 
the return value is checked just a few lines later.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Jaben Carsey 
<jaben.car...@intel.com<mailto:jaben.car...@intel.com>>
------------------------------------------------------------------------------
Sponsored by Intel(R) XDK 
Develop, test and display web and hybrid apps with a single code base.
Download it for free now!
http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to